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

Can't use a password if client certificate is used #579

Closed
future-cyborg opened this issue Mar 15, 2020 · 3 comments · Fixed by #678
Closed

Can't use a password if client certificate is used #579

future-cyborg opened this issue Mar 15, 2020 · 3 comments · Fixed by #678
Labels
support Users asking for help using twine
Milestone

Comments

@future-cyborg
Copy link
Contributor

future-cyborg commented Mar 15, 2020

Bug introduced by #483

I'm trying to upload to a private pypi server. There is a client-certificate SSL reverse proxy in front, and the pypi server has user authentication.

I used to be able to have both of these, but I can't upload anymore since #483.

Looking at #336, I don't think any code change was necessary. Now that the --non-interactive option is available.

Can #483 be reverted please?

@bhrutledge
Copy link
Contributor

@future-cyborg Is this still an issue for you?

@jaraco I think you're closer to the details here; do you have any suggestions?

@bhrutledge bhrutledge added question Discussion/decision needed from maintainers support Users asking for help using twine and removed question Discussion/decision needed from maintainers labels May 27, 2020
@bhrutledge bhrutledge added this to To do in Repository configuration via automation May 29, 2020
@bhrutledge bhrutledge moved this from To do to Support in Repository configuration May 29, 2020
@jaraco
Copy link
Member

jaraco commented May 30, 2020

Thanks for reporting the issue.

Looking at #336, I don't think any code change was necessary. Now that the --non-interactive option is available.

That's interesting. Looks like --non-interactive was introduced in 3.0.

Can #483 be reverted please?

Even if it can be reverted, it shouldn't be summarily reverted. It's important in situations like these to capture all of the requirements in tests. The fact that OP relied on the ability to supply both a password and a client cert was not captured by the tests, which is why this regression in behavior was allowed to occur.

What this project needs instead is a new test on top of the tests added in #483 (possibly tweaked) to capture the desired behavior that captures all of the supported use-cases.

@future-cyborg Would you be willing to put together a pull request that accomplishes that task? And let's try to get the reporters of #336 to confirm that --non-interactive works for their use-case once password support can be disentangled from client cert behavior.

@future-cyborg
Copy link
Contributor Author

Yes, this is still an issue. I've pinned to earlier version to get my functionality.

Yes, I can put together a PR for this. I'll take a look at the the tests cases and see what we can add.

Repository configuration automation moved this from Support requests to Done Aug 16, 2020
@bhrutledge bhrutledge added this to the 3.3.0 milestone Oct 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
support Users asking for help using twine
Projects
Development

Successfully merging a pull request may close this issue.

3 participants