Comments (4)
Users using body streaming on the server side should use Request.BodyStream()
to get an io.Reader
and read as much as they need. It's not recommended to to call Request.Body()
to then get the full body. I agree that we should probably maybe do something here to prevent this. I'm open to a pull request or suggestion on how to fix this.
from fasthttp.
If you want to add a read data size limit to the Body function, can you abandon the Body function and only support BodyStream? BodyStream can satisfy the caller's control of the read data size.
from fasthttp.
@byte0o That would be a backward incompatible change, right?
from fasthttp.
@gab Yes, you can first mark the Body function as deprecated
from fasthttp.
Related Issues (20)
- serving a compress enabled public folder without +w permissions results in a 404 HOT 1
- no free connections available to host HOT 3
- when setting MaxConnsPerIP to value greater than zero, the TLSConnectionState( ) is null on a TLS connection returned from the worker pool. HOT 11
- invalid memory address or nil pointer dereference for firstByteReader'c HOT 2
- peeking nested query args HOT 1
- Incorrect Tag for version 1.54.0 HOT 1
- PFlag/Cobra compatibility in prefork HOT 1
- `\r` is improperly permitted in header names. HOT 1
- How to Optimize the Performance of My fasthttp Client in a Production Environment HOT 1
- `\r` and `\x00` are improperly permitted in header values HOT 1
- fasthttp.FS won't return Content-Encoding when files are too small HOT 4
- Empty header names are improperly permitted HOT 2
- How to get all custom headers of ResponseHeader and RequestHeader HOT 1
- FastHTTP doesn't adequately validate HTTP methods
- SetUserValue from array to map HOT 1
- Type mismatch in AppendUint HOT 9
- Fasthttp Client send multiple origin HOT 2
- FastHTTP doesn't strip tabs from header values
- Reuse body for response HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from fasthttp.