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

Encrypted passwords should not be wrapped #2

Open
mscholl opened this issue Aug 21, 2012 · 1 comment
Open

Encrypted passwords should not be wrapped #2

mscholl opened this issue Aug 21, 2012 · 1 comment
Labels

Comments

@mscholl
Copy link
Member

mscholl commented Aug 21, 2012

If one chooses a sufficiently long password the resulting encrypted password will contain carriage return and line feed as the default Base64 encoding demands cr lf after 76 chars. This may lead to issues when this passwords are read again (if the file where the passwords are store are converted to another encoding etc.). Thus the encrypter should use a Base64 encoding without wrapping.

@ci-cismet-de
Copy link
Member

Integrated branch 'release/2.0-rc5 (d43cb7f)' into 'dev (589e0a9)' -- Build 73

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

2 participants