You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I know this is a semi-old issue, but the good news is that the underlying res.location will change in Express 4.14 to provide better behavior, including no longer throwing when you used your original code.
hi , today i upgrade my node version to 4.4.4 LTS, and i found something wrong in my old project.
the method i am using in my project
the url contains some chinese characters。
,then the error occurs and my program exit.
i find same error in github, http-party/node-http-proxy#964,
in nodejs/node@7bef1b7#diff-286202fdbdd74ede6f5f5334b6176b5cR298,
the commit add a method called _checkInvalidHeaderChar ,it will check the parameter.
when i am using res.redirect ,
i think setHeader should add try catch in https://github.com/expressjs/express/blob/master/lib/response.js#L1046 and
https://github.com/expressjs/express/blob/master/lib/response.js#L718
or using encodeURIComponent because i want to show title of the article in search bar.
The text was updated successfully, but these errors were encountered: