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.

Sign up for GitHub

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

Jump to bottom

ldap: api.environ["LDAP"] is not a string #64

Open
nil0x42 opened this issue Jan 12, 2019 · 1 comment
Open

ldap: api.environ["LDAP"] is not a string #64

nil0x42 opened this issue Jan 12, 2019 · 1 comment
Milestone
Release 3.1

Comments

@nil0x42
Copy link
Owner

nil0x42 commented Jan 12, 2019

This behavior was not expected, but it works from now.
We might consider restricting environment variables setter to only accept strings as values.
If we do so, ldap plugin must be upgraded accordingly.

It might be a problem to allow non string types for environment variables because it makes them uneditable with env core command.
Anyway, i'll have to think about it.
@nil0x42 nil0x42 changed the title ldap plugin uses "LDAP" env var as a dict ldap: api.environ["LDAP"] is not a string Jan 14, 2019
@nil0x42 nil0x42 mentioned this issue Jan 15, 2019
Open
@nil0x42
Copy link
Owner Author

nil0x42 commented Jan 15, 2019

instead of implementing multiline buffers, maintaining current behavior could ease fix of issue #44

@nil0x42 nil0x42 added this to the Release 3.1 milestone Jan 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Release 3.1
Development

No branches or pull requests
1 participant
@nil0x42