Skip to content

[rails] Let thruster handle gzip#619

Open
p8 wants to merge 1 commit intoMDA2AV:mainfrom
p8:rails/thruster-gzip
Open

[rails] Let thruster handle gzip#619
p8 wants to merge 1 commit intoMDA2AV:mainfrom
p8:rails/thruster-gzip

Conversation

@p8
Copy link
Copy Markdown
Contributor

@p8 p8 commented Apr 23, 2026

No description provided.

@p8
Copy link
Copy Markdown
Contributor Author

p8 commented Apr 23, 2026

This fails because compression is only turned on if the response size is greater than 1Kb.
https://github.com/klauspost/compress/blob/v1.17.4/gzhttp/compress.go#L50-L54
The idea is that compression doesn't make sense if the response is smaller than the maximum package size of 1500 bytes.

@MDA2AV
Copy link
Copy Markdown
Owner

MDA2AV commented Apr 23, 2026

Hmm makes sense, I guess for this test we should use larger counts only

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.

2 participants