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 regression when using empty username/password #9850

Merged
merged 2 commits into from
Nov 17, 2024

Conversation

abn
Copy link
Member

@abn abn commented Nov 16, 2024

This change fixes a regression in the password manager that disallowed the use of empty username or password as required by some self-hosted repositories.

Relates-to: #9079 #2538
Resolves: #4181

@abn
Copy link
Member Author

abn commented Nov 16, 2024

We might need to disallow empty usernames.

See: jaraco/keyring#668 jaraco/keyring#687

@abn
Copy link
Member Author

abn commented Nov 16, 2024

Keeping the commits separate.

@abn abn added impact/docs Contains or requires documentation changes area/sources Releated to package sources/indexes/repositories labels Nov 16, 2024
Copy link

github-actions bot commented Nov 16, 2024

Deploy preview for website ready!

✅ Preview
https://website-lwxmmda3o-python-poetry.vercel.app

Built with commit b544ed5.
This pull request is being automatically deployed with vercel-action

@abn abn force-pushed the util-password-manager-empty-creds branch from 69f0e91 to 766aca7 Compare November 16, 2024 23:01
docs/repositories.md Outdated Show resolved Hide resolved
@abn abn force-pushed the util-password-manager-empty-creds branch 2 times, most recently from 940f340 to 29b4102 Compare November 17, 2024 13:18
@abn abn force-pushed the util-password-manager-empty-creds branch from 29b4102 to 640ca13 Compare November 17, 2024 13:56
@abn abn force-pushed the util-password-manager-empty-creds branch from 640ca13 to 18fe382 Compare November 17, 2024 19:38
abn added 2 commits November 17, 2024 21:08
This change fixes a regression in the password manager that disallowed
the use of empty username or password as required by some self-hosted
repositories.

Relates-to: python-poetry#9079 python-poetry#2538
@abn abn force-pushed the util-password-manager-empty-creds branch from 18fe382 to b544ed5 Compare November 17, 2024 20:08
@abn abn requested review from radoering and dimbleby November 17, 2024 20:08
@abn abn merged commit 0c22ca5 into python-poetry:main Nov 17, 2024
74 checks passed
@abn abn deleted the util-password-manager-empty-creds branch November 17, 2024 23:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/sources Releated to package sources/indexes/repositories impact/docs Contains or requires documentation changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't add package from private pypi repository using token access
4 participants