Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify that raw disables both decompression and decoding #439

Merged
merged 3 commits into from
Dec 3, 2024

Conversation

PragTob
Copy link
Contributor

@PragTob PragTob commented Dec 3, 2024

Reading the docs for decoding I was confused like "Wait, what is the difference between :raw and :decode_body then?" And well that is it, searching for :raw more unveils it but I think since the option affects both steps mentioning both effects may be worth it :)

Thanks for your excellent work on req! 💚

Reading the docs for decoding I was confused like "Wait,
what is the difference between `:raw` and `:decode_body` then?"
And well that is it, searching for `:raw` more unveils it but
I think since the option affects both steps mentioning both
effects may be worth it :)
@PragTob
Copy link
Contributor Author

PragTob commented Dec 3, 2024

IMG_20171227_094933_Bokeh

lib/req/steps.ex Outdated Show resolved Hide resolved
lib/req/steps.ex Outdated Show resolved Hide resolved
@wojtekmach wojtekmach merged commit 9d2225c into wojtekmach:main Dec 3, 2024
2 checks passed
@wojtekmach
Copy link
Owner

Thank you!

@PragTob
Copy link
Contributor Author

PragTob commented Dec 3, 2024

lmao you were faster than me I just updated the other bit 😁

@PragTob PragTob deleted the raw-clarification branch December 3, 2024 12:59
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