Skip to content

[PR #1372/2653a469 backport][3.36] Prefer simple JSON when possible - #1373

Merged
gerrod3 merged 1 commit into
3.36from
patchback/backports/3.36/2653a4693f5c9ee3feeda200f66f732f74f4e272/pr-1372
Sep 9, 2026
Merged

[PR #1372/2653a469 backport][3.36] Prefer simple JSON when possible#1373
gerrod3 merged 1 commit into
3.36from
patchback/backports/3.36/2653a4693f5c9ee3feeda200f66f732f74f4e272/pr-1372

Conversation

@patchback

@patchback patchback Bot commented Sep 9, 2026

Copy link
Copy Markdown

This is a backport of PR #1372 as merged into main (2653a46).

A more simple fix vs adding proper q-selection negotiation, all the modern tools want to use JSON so it should be default when possible.

fixes: #1370
Generated-by: gpt-5.6-luna

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

Summary by CodeRabbit

  • Bug Fixes

    • PyPI Simple API requests now prefer JSON when clients advertise both JSON and HTML formats.
    • HTML remains the default when JSON is not explicitly requested.
    • JSON package detail responses now include upload times for every file.
    • Cached responses for JSON and HTML content types are kept separate, preventing one format from affecting the other.
  • Tests

    • Expanded coverage for content negotiation, response fields, and format-specific caching.

fixes: #1370
Generated-by: gpt-5.6-luna
(cherry picked from commit 2653a46)
@patchback patchback Bot mentioned this pull request Sep 9, 2026
4 tasks
@gerrod3
gerrod3 merged commit bc66be8 into 3.36 Sep 9, 2026
14 checks passed
@gerrod3
gerrod3 deleted the patchback/backports/3.36/2653a4693f5c9ee3feeda200f66f732f74f4e272/pr-1372 branch September 9, 2026 17:05
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

Successfully merging this pull request may close these issues.

1 participant