Skip to content

fix windows换行符"\r\n"#591

Open
zysam wants to merge 1 commit into
cnodejs:masterfrom
zysam:patch-1
Open

fix windows换行符"\r\n"#591
zysam wants to merge 1 commit into
cnodejs:masterfrom
zysam:patch-1

Conversation

@zysam

@zysam zysam commented Nov 8, 2015

Copy link
Copy Markdown
Contributor

修复在windows下测试不通过。

修复在windows下测试不通过。
@zysam

zysam commented Nov 8, 2015

Copy link
Copy Markdown
Contributor Author

奇怪,能重新测试吗?

@alsotang

alsotang commented Nov 8, 2015

Copy link
Copy Markdown
Member

这。。。真的存在吗?
windows 在网页上也是 \r\n????

@alsotang

alsotang commented Nov 8, 2015

Copy link
Copy Markdown
Member

我已经让它重跑了,这个肯定是能过的。

@JacksonTian

Copy link
Copy Markdown
Member

👍

@zysam

zysam commented Nov 9, 2015

Copy link
Copy Markdown
Contributor Author

win下本地测试的。也考虑过如果不是本地,会不会有特殊情况,让我再找找。

@alsotang

alsotang commented Nov 9, 2015

Copy link
Copy Markdown
Member

你的 server 是 windows?

@alsotang

alsotang commented Nov 9, 2015

Copy link
Copy Markdown
Member

你看看这东西能不能满足:https://nodejs.org/api/os.html#os_os_eol

@zysam

zysam commented Nov 9, 2015

Copy link
Copy Markdown
Contributor Author

没代理还真上传不了图片。:(

是这样的。我在windows环境下跑测试。

/*
我爱北京天安门<script>alert(1)
</script>
*/

这种多行文本在windows的文件里,肯定是/r/n来换行的。但网页上的,我没确定。

@alsotang

alsotang commented Nov 9, 2015

Copy link
Copy Markdown
Member

在 win 的文件里,你 clone 项目的时候,里面是 \n。你的编辑器会自动把 \n 转成 \r\n??不会的吧

@zysam

zysam commented Nov 9, 2015

Copy link
Copy Markdown
Contributor Author

在 win 的文件里,你 clone 项目的时候,里面是 \n。你的编辑器会自动把 \n 转成 \r\n??不会的吧

会!
escapeSignature 貌似没被使用过 。。。只有用在测试用例上。。晕!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants