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

[question] Support of percent-encoded passwords for tools.Git #10854

Closed
1 task done
beutlich opened this issue Mar 22, 2022 · 3 comments
Closed
1 task done

[question] Support of percent-encoded passwords for tools.Git #10854

beutlich opened this issue Mar 22, 2022 · 3 comments

Comments

@beutlich
Copy link

beutlich commented Mar 22, 2022

Does the Git tool support percent-encoded passwords? It looks to me that it was supported by Conan 1.32.0, but no longer by later versions. Thanks for your clarification.

from conans import tools
import urllib
password = urllib.parse.quote('special!$%', safe='')
git = tools.Git('path/to/repo', username='user', password=password)
@memsharded
Copy link
Member

I don't see at first sight anything evident in code that changed regarding that.

In any case, that tools is being superseded by new from conan.tools.scm import Git, introduced in #10594, docs in conan-io/docs#2419. At this moment, it doesn't have any feature regarding authentication, this is an important thing that needs to be taken carefully. We are going to try to avoid using user/password auth if possible, specially as all Git providers are deprecating it (might still be possible with user/password env-vars, not with the values directly). We are trying to avoid at all costs to have a password leak due to some Conan logging, printing, etc. If you can briefly summarize your needed auth and plans regarding this, it would be useful.

@beutlich
Copy link
Author

beutlich commented Mar 25, 2022

Thanks for the fast reply. I analysed some more details and finally found that 5c09f3e (for 1.33.0) broke our workflow where we already passed percent-encoded passwords to the SCM tool.

@danimtb FYI

@memsharded
Copy link
Member

Yes, apparently it was declared as a bug, and fixed in #8355 (long time ago).

In any case, the feedback over the new integration from conan.tools.scm import Git is appreciated if possible.

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

No branches or pull requests

2 participants