Szymon Łągiewka
41113599af
fix(refactor): prefix built-in node module imports
...
Since v5 relies on node >= 18, this is now possible (since v16, v14.18.0
[^1][^2]).
It's functionally irrelevant:
1. It's not required for CJS nor ESM (with a few exceptions [^3])
2. It has no performance promises
However, there are upsides to this approach:
1. It brings clear boundaries to what's a built-in and what's an
external dependency
2. It reduces the risk of importing unwanted deps where a built-in is
expected
3. It's slightly more interoperable with other JS runtimes that provide
node compatibility[^4], albeit only during development. Once imported
from npm, built-ins are assumed.
[^1]:https://nodejs.org/docs/latest-v22.x/api/modules.html#built-in-modules
[^2]:https://github.com/nodejs/node/pull/37246
[^3]:https://nodejs.org/api/modules.html#built-in-modules-with-mandatory-node-prefix
[^4]:https://docs.deno.com/runtime/fundamentals/node/#using-node's-built-in-modules
2025-01-10 11:53:44 -06:00
Douglas Christopher Wilson
318fd4b543
Merge tag '4.17.3'
2022-02-17 00:27:11 -05:00
Douglas Christopher Wilson
a65913776d
tests: use strict mode
2022-02-09 01:07:08 -05:00
Douglas Christopher Wilson
c7d528cdc0
Merge tag '4.17.2'
2021-12-16 23:01:28 -05:00
drewm
f275e87dff
Fix handling of undefined when "json escape" is enabled
...
closes #4744
2021-11-17 00:00:56 -05:00
Douglas Christopher Wilson
9dd0e7afdb
Fix handling of undefined in res.jsonp
2021-11-16 23:58:17 -05:00
Douglas Christopher Wilson
1b2f3a0698
tests: fix up primitive tests for res.jsonp
2021-11-16 23:53:10 -05:00
Douglas Christopher Wilson
302a6152b4
Merge tag '4.16.4'
2018-10-23 20:38:42 -04:00
Alejandro Estrada
40e04ec7a6
lint: remove usages of "=="
...
closes #3540
2018-02-13 19:30:57 -05:00
Douglas Christopher Wilson
62e12fe710
Merge tag '4.16.2'
2017-10-13 22:27:30 -04:00
Greg Guthe
7154014785
Add "escape json" setting for res.json and res.jsonp
...
closes #3268
closes #3269
2017-09-28 12:03:42 -04:00
Mike Tunnicliffe
ab1c9e924e
Remove res.jsonp(status, obj) signature
...
closes #2940
2017-01-27 23:53:41 -05:00
Douglas Christopher Wilson
1dbfee6623
Merge tag '4.14.0'
2016-06-20 00:37:34 -04:00
Douglas Christopher Wilson
12bc16e72f
tests: use supertest to check response header values
...
closes #2983
closes #2992
2016-05-11 08:50:38 +02:00
Douglas Christopher Wilson
88103063fe
Remove res.jsonp(obj, status) signature
2014-11-06 20:18:42 -05:00
Douglas Christopher Wilson
4ea6f21b02
Merge tag '3.14.0'
2014-07-11 16:28:37 -04:00
Douglas Christopher Wilson
b2382a7336
Remove unnecessary escaping in res.jsonp
2014-07-11 00:20:11 -04:00
Douglas Christopher Wilson
f684a64df7
Add explicit "Rosetta Flash JSONP abuse" protection
2014-07-11 00:15:55 -04:00
Douglas Christopher Wilson
db4a061ed6
Merge tag '3.8.0'
2014-05-21 02:08:04 -04:00
Douglas Christopher Wilson
f14e39d451
set proper charset in content-type for res.send
...
This will write strings to sockets with an explicit "utf8" encoding
(which is the default) and will override the charset in the
Content-Type so it properly relfects the encoding of the response.
closes #1631
closes #2092
2014-05-21 01:31:08 -04:00
Alberto Leal
084f5d891b
Keep previous Content-Type for res.jsonp
...
backport of commit be997fd654
2014-05-21 01:04:29 -04:00
Jonathan Ong
328c6d3060
remove unnecessary test/support/http
...
backport of 643397ed21
2014-05-18 00:57:54 -04:00
Douglas Christopher Wilson
35c50601bd
Merge tag '3.6.0'
2014-05-09 17:33:26 -04:00
Douglas Christopher Wilson
e4302b2120
tests: fixup new json tests
2014-05-08 23:22:18 -04:00
Douglas Christopher Wilson
920f46ad65
tests: add more res.jsonp? tests
2014-05-08 21:27:01 -04:00
Douglas Christopher Wilson
4e1e252e17
deprecate res.json(obj, status)
...
closes #2106
closes #2107
2014-05-08 21:11:05 -04:00
Douglas Christopher Wilson
0bbbc84959
Merge branch '4.1.x'
2014-05-08 14:45:47 -04:00
Douglas Christopher Wilson
2901bd6916
Merge branch '3.5.x' into 4.1.x
2014-05-08 14:01:02 -04:00
Douglas Christopher Wilson
d58ca520c8
Fix res.jsonp error if callback param is object
...
fixes #2104
2014-05-08 13:42:19 -04:00
Alberto Leal
be997fd654
Keep previous Content-Type for res.jsonp
...
closes #2094
2014-05-02 15:01:19 -04:00
Jonathan Ong
bad55f7977
move setHeader charset patch to .set
...
note that application/json no longer adds charset=utf-8. could be a
regression.
closes #1952
See also: https://github.com/broofa/node-mime/issues/86
2014-03-07 16:32:41 -08:00
Jonathan Ong
643397ed21
remove unnecessary test/support/http
2014-03-05 22:06:14 -08:00
Roman Shtylman
8c059469fd
No 'json spaces' by default
...
Json rendering can be handled by user tools or overridden in their own
app to behave as desired. Minimizes the use of magic env settings.
2014-01-11 10:53:36 -05:00
Jonathan Ong
e218377a3d
check existence of jsonp callback
2013-10-15 12:39:32 -07:00
Jon Jenkins
bdbdab7fcc
Fixes from pull request #1643
2013-08-03 16:33:15 -05:00
Guillermo Rauch
64a234958a
fix jsonp whitespace escape. Closes #1132
2012-12-28 10:24:55 -07:00
TJ Holowaychuk
b3936b96e5
add cb && cb(payload) to res.jsonp(). Closes #1374
2012-10-16 11:57:37 -07:00
TJ Holowaychuk
a6caa267bd
add res.jsonp(). Closes #1307
...
this also removes the jsonp
2012-08-30 09:29:43 -07:00
