Comments (11)
👍 I like it.
from got.
👍
P.s. just don't make it silent like in request.
from got.
Hah, I saw that too. Of course not.
from got.
Good idea, 👍
from got.
If someone have ideas, how this option should behave in Stream mode, please share - I can't think nothing better than making it exclusive for callback mode.
from got.
from got.
@floatdrop, by using https://github.com/dominictarr/JSONStream.
from got.
@kevva awesome module, but I tend to leave streaming on feature releases because:
- Streaming JSON is rare
- It needs some additional options (for
parse
andstringify
/stringifyObject
calls) - It will not break compatibility, if we add it later
Now I'm digging through issues with json
-like options in other modules and implementing it right might be harder, than it looks (or I'm just over-complicating).
from got.
Yeah, haven't had a chance to look at it myself, but I agree we can leave this option for streams to future versions.
from got.
After reading some issues with json
option in request one idea came up. All this noise around json
option comes from the fact, that enabling it have two separate effects:
- input - it enables serializing of
body
(and add headers, if they not set) - output - parses incoming JSON
So, why don't just split them apart? For now, if we pass Object
in body
, we will get next error: TypeError: first argument must be a string or Buffer
. What if we serialize it to JSON in this case (without setting any options)? Anyway we can't send it as is, so this seems reasonable. But two things needs to be considered:
- is it true, that most users want to serialize objects in
body
option instead of throwing error? - if so, should we take care of
toString
property as well?
from got.
Partially implemented in 2.7.0
🎉
from got.
Related Issues (20)
- `Response<string>` returned rather than a json response HOT 2
- Is v13 still maintained/supported? HOT 1
- Hangs on revalidated response from cache
- Infinite redirection with special characters in the redirected URL HOT 1
- Support `servername` https option
- How to get traffic usage per request? HOT 1
- Is it possible to skip/disable normalization of a request body? HOT 1
- afterResponse fired 3 times on error HOT 3
- Getting the IP address for failed requests due to timeout
- Post request to API returns response code 500 (Internal Server Error) whereas CURL returns 200 status code HOT 2
- Timeouts are longer than expected HOT 1
- NPM Says: Types are not integrated HOT 1
- FormData getLength can error out when some file input is unknown stream.
- POSTs whose bodys are instances of createReadStream hang with latest node HOT 5
- Apply `applyDestroyPatch` for v11.x
- Stream report downloadProgress when redirects HOT 1
- Is TypeScript support outdated? HOT 1
- Good News for CJS
- afterResponse hook is not called after a retryWithMergedOptions() HOT 2
- HTTP2 with proxy doesn't work? (http2-wrapper)
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 got.