lukechilds/node - node - Gitea: Git with a cup of tea

Author	SHA1	Message	Date
Jeremiah Senkpiel	c9aec2b716	querystring: fix broken stringifyPrimitive stringifyPrimitive has always failed to stringify numbers since its introduction in `422d3c9`. This went uncaught due to encodeURIComponent's string coercion. Fixes: https://github.com/iojs/io.js/issues/1208 PR-URL: https://github.com/iojs/io.js/pull/1213 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Brian White <mscdex@mscdex.net>	10 years ago
Jeremiah Senkpiel	a89f5c2156	querystring: parse numbers correctly Fixes a number parsing regression introduced in `85a92a3` Fixes: https://github.com/iojs/io.js/issues/1208 PR-URL: https://github.com/iojs/io.js/pull/1213 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Brian White <mscdex@mscdex.net>	10 years ago
Brian White	85a92a37ef	querystring: optimize parse and stringify parse optimizations: * Move try-catch to separate function to keep entire function from being deoptimized. * Use key array lookup instead of using hasOwnProperty. * Avoid decoding known empty strings. * Avoid possibly unnecessary switch to slower decoder for values if key decoding throws. stringify optimizations: * Use manual loop for default encoder instead of encodeURIComponent. * Use string concatenation instead of joining an array of strings. * Avoid caching result of typeof. PR-URL: https://github.com/iojs/io.js/pull/847 Reviewed-By: Trevor Norris <trev.norris@gmail.com>	10 years ago
cjihrig	6ac8bdc0ab	lib: reduce util.is() usage Many of the util.is() methods used to check data types simply compare against a single value or the result of typeof. This commit replaces calls to these methods with equivalent checks. This commit does not touch calls to the more complex methods (isRegExp(), isDate(), etc.). Fixes: https://github.com/iojs/io.js/issues/607 PR-URL: https://github.com/iojs/io.js/pull/647 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>	10 years ago
cjihrig	804e7aa9ab	lib: use const to define constants This commit replaces a number of var statements throughout the lib code with const statements. PR-URL: https://github.com/iojs/io.js/pull/541 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>	10 years ago
isaacs	3e1b1dd4a9	Remove excessive copyright/license boilerplate The copyright and license notice is already in the LICENSE file. There is no justifiable reason to also require that it be included in every file, since the individual files are not individually distributed except as part of the entire package.	10 years ago
Ben Noordhuis	21130c7d6f	lib: turn on strict mode Turn on strict mode for the files in the lib/ directory. It helps catch bugs and can have a positive effect on performance. PR-URL: https://github.com/node-forward/node/pull/64 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Fedor Indutny <fedor@indutny.com>	10 years ago
Tristan Berger	0f2956192c	querystring: fix unescape override Documentation states that `querystring.unescape` may be overridden to replace unescaper during parsing. However, the function was only being used as a fallback for when the native decoder throws (on a malformed URL). This patch moves the call to the native function and the try/catch around it into querystring.unescape then has the parser always invoke it, so that an override will always be used. Fixes #4055 Reviewed-By: Fedor Indutny <fedor@indutny.com>	10 years ago
cjihrig	61ddad1314	querystring: do not add sep for empty array Currently, stringification of an empty array outputs a single separator character. This commit causes an empty array to output the empty string. Signed-off-by: Fedor Indutny <fedor@indutny.com>	11 years ago
fengmk2	4729202d1e	querystring: custom encode and decode Not all querystring are utf-8 encoding, make querystring can be used to encode / decode `non-utf8` encoding string if necessary. Signed-off-by: Timothy J Fontaine <tjfontaine@gmail.com>	11 years ago
Yorkie	8d3bc88bbe	querystring: remove `name` from `stringify()` QueryString.stringify() allowed a fourth argument that was used as a conditional in the return value, but was undocumented, not used by core and always was always false/undefiend. So the argument and conditional have been removed. Signed-off-by: Trevor Norris <trev.norris@gmail.com>	11 years ago
Trevor Norris	87cde44280	Revert "util: more strict check for bool/number/string" This reverts commit `95ee84fabe`.	11 years ago
Yorkie	95ee84fabe	util: more strict check for bool/number/string	11 years ago
isaacs	22c68fdc1d	src: Replace macros with util functions	12 years ago
Ben Noordhuis	0330bdf519	lib: macro-ify type checks Increases the grep factor. Makes it easier to harmonize type checks across the code base.	12 years ago
Mitar	e5b90a14b8	querystring: Removing unnecessary binding Binding of `http_parser` in querystring isn't used anywhere and should be removed.	12 years ago
isaacs	f105f2f2fd	trivial: Doc typo and lint fix	13 years ago
Felix Böhm	3a5798b097	querystring: improved speed and code cleanup	13 years ago
isaacs	a811a4a130	Fix #3058 querystring: Fix incorrect handling of empty keys	13 years ago
isaacs	d3b8372615	lint, missed in QS speed merge	13 years ago
Brian White	5e3ca98155	Make QueryString.parse() even faster	13 years ago
isaacs	0cdf85e28d	Lint all the JavaScripts.	13 years ago
Philip Tellis	5166758927	Make QueryString.parse run faster Use decodeURIComponent when appropriate, and only fall back to querystring.decode if it throws, or if the character is a '+'. Fix #2248	13 years ago
Fedor Indutny	23de33968f	querystring: fix maxKeys = 0 is ignored	13 years ago
Fedor Indutny	8a98c2f1d8	http, querystring: added limits to prevent DoS	13 years ago
isaacs	e06ce7562c	Fix #1707 hasOwnProperty usage If hasOwnProperty is overridden, then calling `obj.hasOwnProperty(prop)` can fail. Any time a dictionary of user-generated items is built, we cannot rely on hasOwnProperty being safe, so must call it from the Object.prototype explicitly.	13 years ago
isaacs	98990b9779	Fix #1707 hasOwnProperty usage	13 years ago
isaacs	ac4791393e	Fix #1497 querystring: Replace 'in' test with 'hasOwnProperty'	14 years ago
isaacs	3210809d0a	Fix #1497 querystring: Replace 'in' test with 'hasOwnProperty'	14 years ago
Ryan Dahl	55048cdf79	Update copyright headers	14 years ago
Jeremy Martin	6f726cf8c7	url.parse(url, true) defaults query field to {}	14 years ago
Ryan Dahl	e232f6e735	more lint	14 years ago
Ryan Dahl	6ff12c425e	Add querystring.unescapeBuffer	14 years ago
Ryan Dahl	57d8172906	query-string unescape moved to js land	14 years ago
Peter Griess	422d3c93bc	Get rid of PHP/Rails style parameter munging. - Handle only the most basic of query string parsing and construction. Leave the rest (e.g. Rails/PHP behaviors) to modules higher up the stack, like Express.	14 years ago
cloudhead	fe3e419687	querystring.parse: handle undefined value properly	15 years ago
Micheil Smith	299671b731	querystring: Removing typecasting of numeric strings to numbers The tests did not accurately test for a strict equality, meaning that the number == to the string.	15 years ago
Jan Kassens	bb2acd5e75	querystring.stringify is now more solid * handles NaN and Infinity * works with arrays from other contexts	15 years ago
Dmitry Baranovskiy	f8ca6b383c	Rewrote QueryString.parse to make it smaller and more effective. Also added ability to parse foo.bar=4 equal to foo[bar]=4 Added tests for this as well	15 years ago
Dmitry Baranovskiy	8ec12339f5	Refactored isA, isBool, etc functions to use some of ES5 goodness.	15 years ago
isaacs	4ce100fa62	A replacement for decodeURIComponent that doesn't throw. And add a few more tests.	15 years ago
Thomas Lee	a63ce5cd72	Allow callers to disable PHP/Rails style parameter mungeing in querystring.stringify	15 years ago
Tim Caswell	62d9852c3d	Replace slow and broken for..in loops with faster for loops over the keys.	15 years ago
isaacs	57fbb627ca	trailing whitespace fixes	15 years ago
Ryan Dahl	b9cd3363ef	Bugfix in querystring	15 years ago
isaacs	7ff04c1f86	Add URL and QueryString modules, and tests for each. Also, make a slight change from original on url-module to put the spacePattern into the function. On closer inspection, it turns out that the nonlocal-var cost is higher than the compiling-a-regexp cost. Also, documentation.	15 years ago