Skip to content

Commit

Permalink
fixup! http: disallow two-byte characters in URL path
Browse files Browse the repository at this point in the history
  • Loading branch information
bennofs committed Nov 22, 2017
1 parent eab523c commit 58ac26f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/_http_client.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ const errors = require('internal/errors');
// with an additional rule for ignoring percentage-escaped characters, but
// that's a) hard to capture in a regular expression that performs well, and
// b) possibly too restrictive for real-world usage. So instead we restrict the
// filter to just control characters and spaces.
// filter to just control characters, spaces and two-byte characters.
//
// This function is used in the case of small paths, where manual character code
// checks can greatly outperform the equivalent regexp (tested in V8 5.4).
Expand Down

0 comments on commit 58ac26f

Please sign in to comment.