fix proxy statusCode - Githubissues

goddyZhao / nproxy

A cli proxy tool specialized in file replacing

http://goddyzhao.me/nproxy

MIT License

510 stars 87 forks source link

Closed xiaojue closed 10 years ago

xiaojue commented 10 years ago

这里应该用代理请求返回的statuscode，之前写死成200了。导致304请求会变空白，因为body没东西输出。。

cgcgbcbc commented 10 years ago

@xiaojue 似乎很久联系不上作者了也不知道什么时候才能merge进来诶

xiaojue commented 10 years ago

@cgcgbcbc 还好，我上次的pull，他1个月就merge了。。

cgcgbcbc commented 10 years ago

@xiaojue 上次我给作者发邮件说有pull request需要他处理...然后他merge了你的，然后我的pr他指出了问题之后我过了一阵子改了然后再也联系不上了...

xiaojue commented 10 years ago

看了一下，果然我提交之前就是写死的200。你的那个pr，我刚看了一下。。

不用+1吧。如果listen的是0，系统会自动分配一个没占用的端口给你，你再监听listening事件，拿到port就可以了。。

我也同意作者给你的修改意见。不过看了代码你还是+1了。。。？

https://github.com/SinaBlogFE/fd-server/blob/master/lib/utils.js#L9 我这里有一个获取随机端口的，需求是我要提前知道port。

如果不需要提前知道，只想随机，直接listen 0 就行。

@cgcgbcbc

cgcgbcbc commented 10 years ago

看了一下，果然我提交之前就是写死的200

我的意思是他上次处理pull request是我发邮件告诉他有pull request要处理然后他才来github上看的，貌似作者把notification关掉了的样子@_@

不过看了代码你还是+1了。。。？

囧...貌似当时不清醒了..那行应该去掉...然后就是listen(0)会随机分配端口，然后在on listening里保存系统分配的端口...晚上我再force update一下pull request...

xiaojue commented 10 years ago

@cgcgbcbc 你头像很像我以前一个同事啊。。恩。。不清醒，喝多了的感觉……

cgcgbcbc commented 10 years ago

@xiaojue how about contact Mr. Zhao and add us as collaborators to this repo..

goddyZhao commented 10 years ago

Thanks for your fix @xiaojue and sorry about the loooong delay!

xiaojue commented 10 years ago

@goddyZhao You're welcome 霍霍