tencentyun / wafer2-node-sdk

Wafer2 SDK for Node.js
Other
268 stars 57 forks source link

增加serverPort配置。兼容不使用80端口的服务 #29

Closed miwoy closed 6 years ago

miwoy commented 6 years ago

同时也兼容之前配置,不影响之前任何功能,我本地跑项目时,不想使用80端口

jas0ncn commented 6 years ago

本地测试是没办法使用信道服务的。。不知修改这个有何用处呢?

jas0ncn commented 6 years ago

另外一点,Node.js 程序一般不是直接开在 80 端口的,需要 Nginx 反向代理一下 80 端口。如果这样修改,回调地址就错了。

miwoy commented 6 years ago

我自己写了一个信道服务,所以本地是可以测试的 然后,这个配置不是强制性的,如果不配置和原来一样并不影响 一般开发和部署会使用两个配置文件,所以如果开发人员明白这个配置是干嘛的,就不会出现回调地址错了的问题。

miwoy commented 6 years ago

当然我的目的确实是给本地测试用的,不过你也说是一般是使用nginx代理到80端口,那些不一般的人如果不想使用80的情况为啥不能兼容一下

jas0ncn commented 6 years ago

目前这个 SDK 还是建议是使用我们的信道服务,不过可以 merge 一下你这 pr。