Skip to content

fix: replace python-dxf with direct httpx for manifest Content-Type#19

Merged
davidpoblador merged 1 commit intomainfrom
worktree-valiant-bubbling-horizon
Apr 2, 2026
Merged

fix: replace python-dxf with direct httpx for manifest Content-Type#19
davidpoblador merged 1 commit intomainfrom
worktree-valiant-bubbling-horizon

Conversation

@davidpoblador
Copy link
Copy Markdown
Member

Summary

  • Reverted python-dxf adoption in favor of direct httpx calls. python-dxf's set_manifest doesn't pass through Content-Type, causing 400 errors when pushing OCI image index manifests (application/vnd.oci.image.index.v1+json) upstream.
  • Fixed HTTP redirect safety: follow_redirects only enabled on GET/HEAD to prevent POST-to-GET downgrade on 301/302 redirects.
  • Added upper bound to uv_build version requirement (>=0.11.3,<0.12).

Test plan

  • All 61 tests pass, including new test_push_manifest_oci_index that validates OCI content-type passthrough
  • Deploy and verify upstream sync succeeds for OCI image index manifests

🤖 Generated with Claude Code

… manifest push

python-dxf's set_manifest doesn't pass through the Content-Type header,
causing 400 errors when pushing OCI image index manifests upstream. Switch
back to direct httpx calls with proper Content-Type handling.

Also fixes HTTP redirect safety (follow_redirects only on GET/HEAD to
prevent POST-to-GET downgrade) and adds upper bound to uv_build version.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@davidpoblador davidpoblador merged commit 9a9d358 into main Apr 2, 2026
1 check passed
@davidpoblador davidpoblador deleted the worktree-valiant-bubbling-horizon branch April 2, 2026 10:46
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