-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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 handling errors 401, 403 when looking for repositories #3337
Conversation
My experience of our pypicloud returning an
EDIT: Read more code and realized authentication is not related to this. |
0457818
to
e9e0d93
Compare
Looks like the exception is raised manually, below your conditional, so I believe I spoke in error. |
What's the status on this PR? |
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.
Thank you for your contribution @cereblanco. This might require a rebase as well.
Co-authored-by: Arun Babu Neelicattu <arun.neelicattu@gmail.com>
515ed6b
to
a7f0ceb
Compare
a7f0ceb
to
41d0a2b
Compare
Hi @cereblanco this PR still needs to be rebased because of the breaking CI build. Thank you in advance! |
Hi 🤚 How can I get information about when the new version with this change will be released? |
@erdemuysal I faced the same issue. They approved the MR for poetry 1.2.0 release Currently it is available only as an alpha release. It solved my problem. |
Is there a reason not to back port this to 1.11? @stephsamson |
@abn @sdispater Could this simple and non-intrusive fix be backported to |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Pull Request Check List
fixes error handling related to: #3303