Skip to content

(v6.x backport) http: describe parse err in debug output #14416

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.

Sign up for GitHub

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

Closed
sam-github wants to merge 14 commits into from
Closed

(v6.x backport) http: describe parse err in debug output #14416

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
2fb1381
doc: document and test that methods return this
sam-github Jun 7, 2017
dce4a75
test: mark test-fs-readdir-ucs2 flaky
joaocgreis Jun 29, 2017
448c8d4
test: mark test-npm-install flaky on arm
refack Jul 2, 2017
3b2c791
build: run test-hash-seed at the end of test-v8
targos Jul 13, 2017
ddc8050
cluster, dns, repl, tls, util: fix RegExp nits
vsemozhetbyt Jun 7, 2017
0b17a45
net: support passing undefined to listen()
sam-github Jul 14, 2017
424fad5
doc: prepare js code for eslint-plugin-markdown
vsemozhetbyt Apr 21, 2017
19ff807
doc: conform to rules for eslint-plugin-markdown
vsemozhetbyt Apr 21, 2017
ce4718f
build, doc, tools: add eslint-plugin-markdown
vsemozhetbyt Jul 4, 2017
05b95fa
tools: remove no-useless-regex-char-class-escape
Trott Jan 1, 2017
c52f60f
tools: fix indentation in required-modules.js
Trott Jun 18, 2017
4d91ff4
tools: apply stricter indentation rules to tools
Trott Jun 18, 2017
ecc50e9
tools: fix error in custom ESLint rule
Trott Jun 19, 2017
cf8045a
http: describe parse err in debug output
sam-github May 24, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
tools: fix error in custom ESLint rule 
Fix previously-unnoticed typo in `required-modules.js`.

Backport-PR-URL: #14360
Refs: #13758 (comment)
PR-URL: #13758
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
  • Loading branch information
@Trott @MylesBorins
Trott authored and MylesBorins committed Jul 21, 2017
commit ecc50e9d912573eec108e143b561df219b46e183
2 changes: 1 addition & 1 deletion tools/eslint-rules/required-modules.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ module.exports = function(context) {
if (foundModules.length < requiredModules.length) {
var missingModules = requiredModules.filter(
function(module) {
return foundModules.indexOf(module === -1);
return foundModules.indexOf(module) === -1;
}
);
missingModules.forEach(function(moduleName) {
Expand Down