某些变态的WEB服务器不认全小写的请求头, 特别是Cookie这个字段. #669

Closed
wants to merge 2 commits into
from

Projects

None yet

7 participants

@youxiachai

Please use english .....

@xqin

@youxiachai 楼上你看不懂?
即便不认识汉字, 看图片和diff,也知道是改的什么和为什么要改的.

PS: 本人英文不行, 如果你行的话,麻烦你翻译一下,造福大众.

@denys

@xqin ingenious troll.

@denys

@xqin @youxiachai BTW: code says it all.

@meteormatt

这bug报的👎

@sxyizhiren

request的cookie总觉得不够强大

@xqin

@sxyizhiren request的cookie使用的 cookie-jar 这个模块,而这个模块对Cookie的处理在get cookie时根本不区分 域名. 而有人给作者pull request了, 但作者好像很长时间没登陆github了,一直没合并请求.
Example:

var jar = new Jar;
var a = new Cookie('sid=1111; path=/foo/bar; domain=bar.com');
jar.add(a);

var cookies = jar.get({ url: 'http://foo.com/foo/bar' });

console.log(cookie.length);
console.dir(cookie[0]);

@youxiachai

cookie-jar author is @mikeal ..

@sxyizhiren

@xqin
它的cookie我也提了个issue,#670 .
另外我fork了分支,替换掉了cookie模块,已经能够实现domain的区分。https://github.com/sxyizhiren/request-5291 .

@xqin

@sxyizhiren 麻烦你帮忙把我中文翻译一下, request 的作者好像不管中文提交的合并请求.

这个人提的(他只提了Authorization )比我晚, 都合并了,我半年前都提了,不给我合并.
#804

而且我看目前最新的request.js 里面在一些setHeader中还是全小写的key.

@sxyizhiren

@xqin cookie模块已经改了,作者已经合并了新的cookie模块。区分域名的问题解决了,不过大小写的问题我不知道怎么样。
你的大小写问题的翻译:some web server don't accept cookie field if the word "cookie:" was all lowercase.

@xqin

@sxyizhiren 谢谢, 我刚刚从原来的分支上反合了一次代码,并更新为最新的,然后修改了 setHeader方法(Line:1035), 自动将HTTP请求中的header的name值转换为 Pascal-Case 风格. :)
就看作者合不合了.

@LoicMahieu
request member

Speak english on Github please.

@LoicMahieu LoicMahieu closed this Jul 10, 2014
@andyhu

The author says no :) 作者确定不会改这个问题,可以自己在本地patch,一样的
#830

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment