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 SCM by making user and password url-encoded #8355
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only one question
return "{scheme}://{user}:{password}@{url}".format(scheme=scheme, | ||
user=quote_plus(the_user), | ||
password=quote_plus(the_password), | ||
url=url) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
other schemes like file
or git
should execute quote_plus
too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
file
format is excluded in this return https://github.com/conan-io/conan/pull/8355/files#diff-dae658b6639940b7fd00c86a6ce57086f89c33ed5ae3a40a9a91f8ac49c12cd4R77, not sure for git
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apparently it is not supported in the URL https://stackoverflow.com/questions/31801271/what-are-the-supported-git-url-formats and git protocol has no authentication support https://stackoverflow.com/a/33846897/9585455
Changelog: BugFix: Fix SCM user and password by making them url-encoded
Docs: omit
develop
branch, documenting this one.Note: By default this PR will skip the slower tests and will use a limited set of python versions. Check here how to increase the testing level by writing some tags in the current PR body text.