-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Support huge read-only []byte response bodies #477
Support huge read-only []byte response bodies #477
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you maybe add some test cases to verify that everything works. And so we can keep it working in the future as well when things change.
@cipriancraciun would it be possible for you to add some tests to make sure everything keeps working correctly in the future? |
Sorry for deferring this, but I was a little busy. I'll try to add them during these days around new year's eve. |
… touching it (as an alternative to `SetBody`)
… incoduction of `Response.bodyRaw`
29a2b11
to
cd8edeb
Compare
So I've rebased my two commits based on the latest code, and added a few tests at the end of If I can make any other changes please let me know. |
Thanks. |
See issue #469.