Skip to content

strip transfer encoding#820

Merged
bearsyankees merged 2 commits into
mainfrom
devin/1784598370-strip-transfer-encoding
Jul 21, 2026
Merged

strip transfer encoding#820
bearsyankees merged 2 commits into
mainfrom
devin/1784598370-strip-transfer-encoding

Conversation

@bearsyankees

Copy link
Copy Markdown
Collaborator

No description provided.

@greptile-apps

greptile-apps Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR normalizes HTTP framing when replaying captured requests. The main changes are:

  • Removes inherited Content-Length and Transfer-Encoding headers case-insensitively.
  • Recomputes Content-Length from the plain replay body.
  • Adds a test for mixed-case Transfer-Encoding: chunked removal.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • The updated framing logic matches the function's plain-body serialization behavior.
  • The new test covers case-insensitive header removal and length recomputation.

Important Files Changed

Filename Overview
strix/tools/proxy/caido_api.py Removes stale framing headers before serializing a plain, non-chunked replay body.
tests/test_proxy_client.py Adds coverage for removing mixed-case Transfer-Encoding while recomputing Content-Length.

Reviews (1): Last reviewed commit: "test(proxy): drop transfer encoding on r..." | Re-trigger Greptile

@bearsyankees
bearsyankees merged commit f9890a6 into main Jul 21, 2026
2 checks passed
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