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

Fix #509 #510

Merged
merged 7 commits into from
Dec 15, 2017
Merged

Fix #509 #510

merged 7 commits into from
Dec 15, 2017

Conversation

micafer
Copy link
Member

@micafer micafer commented Dec 15, 2017

No description provided.

@micafer micafer merged commit f3bc2ba into master Dec 15, 2017
@jartitodebuscarusuario
Copy link

For uniformity reasons, maybe this change should be extended to all fields in the authorization file, i.e., all field values should be enclosed by single quotes.

Regards

@jartitodebuscarusuario
Copy link

Sorry, I have seen that this change is already implemented for all fields, not only passwords.

Regards

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