Skip to content

Add uploader next to author/maintainer in info json #10575

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MartinThoma
Copy link

@MartinThoma MartinThoma commented Jan 9, 2022

I've noticed that on https://pypi.org/project/identifier/ I can see the user "Collie" as maintainer, but on https://pypi.org/pypi/identifier/json the user cannot be seen at all.

It would be nice to be able to get the same information in a structured way as via the GUI.

I'm uncertain if the uploader email is kept confidential.

Copy link
Member

@miketheman miketheman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uploader value is not guaranteed to be available, per https://github.com/pypa/warehouse/blob/bc54b9db85e331623abe8b51c5954bf1de688c59/warehouse/packaging/models.py#L477

Have you considered how to guard against that possibility?

Adding a test to cover this behavior in https://github.com/pypa/warehouse/blob/main/tests/unit/legacy/api/test_json.py

And specifically using the ReleaseFactory to create a release without an uploaded value would be the way to go.

@miketheman miketheman added the awaiting-response PRs and issues that are awaiting author response label Jan 9, 2023
@MartinThoma MartinThoma requested a review from a team as a code owner February 22, 2024 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-response PRs and issues that are awaiting author response
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants