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

include_ldap_query parameter attrs not valid (6.2.17b1) #22

Closed
morenonardelliunitn opened this issue Jun 9, 2017 · 9 comments
Closed
Labels

Comments

@morenonardelliunitn
Copy link

Hi to all,
my problem is the - caracter in the filed attrs on the include_ldap_query.
In the last version of sympa (6.1.x) my workaround is changed the value of the List.pm at line:

'include_ldap_query' => {'format' => {'host' => {'format...
...
'attrs' => {'format' => '\w+',

to:

'include_ldap_query' => {'format' => {'host' => {'format...
...
'attrs' => {'format' => '.+',

but now???? This problem blocking my organization....

@racke
Copy link
Contributor

racke commented Jun 9, 2017

Sorry, I can't find this piece of code. Please either state the file and line number or include it in the issue as

code block

@ikedas
Copy link
Member

ikedas commented Jun 9, 2017

@MorenoUniTN, what lind of attribute name you want to enter? Could you show example?

@ikedas
Copy link
Member

ikedas commented Jun 11, 2017

@MorenoUniTN, does the change above fix the problem?

@morenonardelliunitn
Copy link
Author

@racke This part of code is refered to fle List.pm of Sympa version 6.1 the new List.pm of Sympa 6.2 is complety differents and not contain this part of code

@ikedas Yes, in the old version of Sympa 6.1 this workaround fix the problem

there is a place in the new sympa 6.2 to set enable the caracter - in attrs parameter?
Thanks to all

@ikedas
Copy link
Member

ikedas commented Jun 13, 2017

I'll merge PR #23. I feel appreciate if you check the next beta.

@morenonardelliunitn
Copy link
Author

@ikedas Thanks for all, but my respons above is refered to old code of old version sympa 6.1.x
I try to modify file indicated at #23 but i have the same error.
The caracter is - dash in the attrs value of ldap query
screenshot 2017-06-14 at 08 53 38

@ikedas
Copy link
Member

ikedas commented Jun 14, 2017

@MorenoUniTN, can you apply this patch to new version?
https://github.com/sympa-community/sympa/pull/23.patch

@morenonardelliunitn
Copy link
Author

@ikedas Great! now it's ok.
thanks a lot for all!!!

@ikedas
Copy link
Member

ikedas commented Jun 14, 2017

Good. So this issue will be closed. Thank you for reporting bug!

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