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

Error processing scenario #813

Closed
BrunoMalaval opened this issue Nov 27, 2019 · 5 comments · Fixed by #814
Closed

Error processing scenario #813

BrunoMalaval opened this issue Nov 27, 2019 · 5 comments · Fixed by #814
Labels
Milestone

Comments

@BrunoMalaval
Copy link

Version

6.2.49b.1

Installation method

Source package

Expected behavior

Fix parsing scenario

Actual behavior

In a create scenario (create_list.uha) , I use 2 ldap search filters to control creation list

Scenario is

is_listmaster([sender])     md5,smime -> do_it
equal([sender],'dnum.listmaster@uha.fr')        md5,smime -> do_it
search(membres_di.ldap)     md5,smime -> do_it
search(personnel_uha.ldap)  smtp,md5,smime -> listmaster,notify
true()                      smtp,md5,smime -> reject(reason='create_list_local_user')

On the home page, I get this error

DIED: /usr/local/sympa/etc/scenari/create_list.uha: Bareword "membres_di" not allowed while "strict subs" in use at (eval 177) line 17. Bareword "ldap" not allowed while "strict subs" in use at (eval 177) line 17. Bareword "personnel_uha" not allowed while "strict subs" in use at (eval 177) line 20. Bareword "ldap" not allowed while "strict subs" in use at (eval 177) line 20. 
 at /usr/local/sympa/bin/Sympa/Scenario.pm line 249.
Sympa::Scenario::compile('lists-diff.uha.fr', 'title.gettext Demande de creation de liste UHA\x{a}\x{a}is_listmaster...', 'function', 'create_list', 'file_path', '/usr/local/sympa/etc/scenari/create_list.uha') called at /usr/local/sympa/bin/Sympa/Scenario.pm line 218
Sympa::Scenario::new('Sympa::Scenario', 'lists-diff.uha.fr', 'create_list') called at /usr/local/sympa/bin/wwsympa.fcgi line 2697
main::check_param_in() called at /usr/local/sympa/bin/wwsympa.fcgi line 1503

I test to remove the first search() , and get the same error for the second

Additional information

Thanks a lot
Bruno

@racke
Copy link
Contributor

racke commented Nov 27, 2019

I suppose this works in earlier versions of Sympa?

@BrunoMalaval
Copy link
Author

Yes,
It works fine on my production server (6.2.16)
And tested on a 6.2.44 with no error.

It looks like an error in parsing scenario, not when process it

@racke
Copy link
Contributor

racke commented Nov 27, 2019

The module has been recently rewritten (#782), so it looks like a regression. Thanks for testing the beta version and reporting the problem!

@racke racke added the bug label Nov 27, 2019
@racke racke added this to the 6.2.50 milestone Nov 27, 2019
@ikedas ikedas mentioned this issue Nov 27, 2019
@ikedas
Copy link
Member

ikedas commented Nov 27, 2019

Hi @BrunoMalaval ,

Could you please apply this patch and check if the problem will be solved? Thanks.

@BrunoMalaval
Copy link
Author

Hi @ikedas

It fix the error, so I can connect and will continue my tests

Thanks a lot, great job

ikedas added a commit that referenced this issue Dec 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants