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

add config option for an authorization file #7

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

wberrier
Copy link

@wberrier wberrier commented Jan 4, 2017

in case the credentials should be loaded from an external file

My particular use case is that I want to version control the .cfg
file, but I don't want to put my username/password in the file

wberrier and others added 5 commits January 4, 2017 16:51
in case the credentials should be loaded from an external file

My particular use case is that I want to version control the .cfg
file, but I don't want to put my username/password in the file
Mostly in regards to unicode string handling.

Ldap search results returns "keys" in utf-8, but values are often in
bytes. stringify values after querying so that results and cache
aren't treated differently.

Open files in text mode instead of binary mode.

SafeConfigParser was renamed to ConfigParser in python3.

LOG.warn is deprecated, change to "LOG.warning" to avoid deprecation
output messages.
... such as pass from the passwordstore
allow setting password by passing command
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants