“setting”页面里的“个人网站”选项不支持"https://"

[meteormatt]

设置页面里的“个人网站”选项不支持https://，
只支持"http://"

bug重现：

1. 我在个人网站里填入https://meteormatt.net
   
2. 保存时提示

不正确的个人网站。

我看了下代码，好像是这行报的错。
https://github.com/cnodejs/nodeclub/blob/master/controllers/user.js#L129

if (url !== '') {
      try {
        if (url.indexOf('http://') < 0) {
          url = 'http://' + url;
        }
        check(url, '不正确的个人网站。').isUrl();
      } catch (e) {

这段代码没有检查https://的情况。
如果是"https://"，index<0，反而变成了"http://https://"的情况，造成异常发生。

希望这个bug能够得到修复。
不然，我自己提交一个pull request，增加一个or判断？

[fengmk2]

非常欢迎提交pr
On Mar 21, 2013 11:34 PM, "Liu Xing" notifications@github.com wrote:

设置页面里的“个人网站”选项不支持https://，
只支持"http://"

bug重现：

1. 我在个人网站里填入https://meteormatt.net
   [image: 2013-03-21 22:55:26]https://f.cloud.github.com/assets/424475/286282/750cf7be-9238-11e2-8982-71cd750757eb.png
2. 保存时提示

不正确的个人网站。

[image: 2013-03-21 22:54:20]https://f.cloud.github.com/assets/424475/286283/7b07ec3c-9238-11e2-9310-f49620362d54.png

我看了下代码，好像是这行报的错。
https://github.com/cnodejs/nodeclub/blob/master/controllers/user.js#L129

if (url !== '') {
try {
if (url.indexOf('http://') < 0) {
url = 'http://' + url;
}
check(url, '不正确的个人网站。').isUrl();
} catch (e) {

这段代码没有检查https://的情况 https://xn--87qz08afnq。
如果是"https://"，index<0，反而变成了"http://https://"的情况，造成异常发生。

希望这个bug能够得到修复。
不然，我自己提交一个pull request，增加一个or判断？

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/132
.

[meteormatt]

测试通过，issue关闭。