Skip to content

Conversation

@fatboypunk
Copy link
Contributor

This was missing, and when the encoding is gzip, we don't get parsed_body as a result anymore

  • Review required
  • Includes 1+ tests
  • Fully tested locally

@fatboypunk fatboypunk requested a review from MvanDiemen April 30, 2025 12:27
Copy link
Member

@MvanDiemen MvanDiemen left a comment

Choose a reason for hiding this comment

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

Nice :D

This was missing, and when the encoding is gzip, we don't get parsed_body as a result anymore
@fatboypunk fatboypunk force-pushed the marcel/convert-request branch from 17ac3ca to 3646718 Compare April 30, 2025 12:36
@fatboypunk fatboypunk force-pushed the marcel/convert-request branch from 4e1259b to e3652b5 Compare April 30, 2025 12:40
@fatboypunk fatboypunk merged commit c768692 into main Apr 30, 2025
2 checks passed
@coveralls
Copy link

Pull Request Test Coverage Report for Build e3652b5c4cac48048797f9aceb2ed61b6a67a5b6-PR-17

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 85.634%

Totals Coverage Status
Change from base Build 65e5257d34e5c789abf0f83666d2b1b3630495bf: 0.0%
Covered Lines: 459
Relevant Lines: 536

💛 - Coveralls

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.

4 participants