Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extra space at the end of line in topics.conf #581

Closed
ikedas opened this issue Mar 31, 2019 · 3 comments
Closed

Extra space at the end of line in topics.conf #581

ikedas opened this issue Mar 31, 2019 · 3 comments
Labels
Milestone

Comments

@ikedas
Copy link
Member

ikedas commented Mar 31, 2019

Version

6.2.40

Installation method

RPM (Sympa-JA.org)

Actual behavior

If topics.conf has this content:

other
visibility conceal␣

( is a space)

Each time the users access to web interface, this error message is shown in the log:

Unable to find scenario file "topics_visibility.conceal "
@ikedas ikedas added the bug label Mar 31, 2019
@dverdin
Copy link
Contributor

dverdin commented Apr 3, 2019

To fix this, we might change Conf.pm to ignore trailing spaces.

@racke
Copy link
Contributor

racke commented Apr 3, 2019

Yeah good idea. As a rule, always strip surrounding spaces.

@dverdin
Copy link
Contributor

dverdin commented Apr 3, 2019

Damn. We seem to be doing it already. It was too easy...

$value =~ s/\s*$//;

@ikedas ikedas changed the title Extra space at hte end of line in topics.conf Extra space at the end of line in topics.conf Apr 6, 2019
@ikedas ikedas added this to the 6.2.44 milestone May 24, 2019
@ikedas ikedas closed this as completed in 2ceea13 Jun 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants