vue-router路径计算问题

昨天刚刚发表了一个前端跨域新方案尝试，今天在开发中就遇到的了问题。http://www.jianshu.com/p/56e1500e4060

起因

前端使用的是vue-router组件的history模式，但是由于我们的整个页面都是从static（静态资源站）load过来的，所以其他页面自然也需要跨域去拿，然而就在跨域的时候 vue-router 出了问题。

分析问题

我们的api站点在 api.com
而静态资源在 static.com，页面的base标签也指向static

<base href="http://static.com" />

然而，在访问 test模板时却跳到了http://api.com/http:/static.com/test

经过一些简单的断点调试，锁定了以下代码

vue-router.esm.js

//1794行~1675行function normalizeBase (base) {  if (!base) {    if (inBrowser) {      // respect <base> tag      var baseEl = document.querySelector('base');      base = (baseEl && baseEl.getAttribute('href')) || '/';    } else {      base = '/';    }  }  // make sure there's the starting slash  if (base.charAt(0) !== '/') {    base = '/' + base;  }  // remove trailing slash  return base.replace(/\/$/, '')}

这段代码的作用是设置路由的base路径，如果有兴趣一路跟踪的话会发现这个base参数是由实例化VueRouter时候传入的options.base；

再看代码，他会判断如果base是空的，那么就会给一个默认值：
如果实在浏览器（非服务器环境）下执行，那么会调用document.querySelector('base')来尝试获取<base href='' />标签中href属性的值；

在我们实际的场景中，这里得到一个跨域的绝对地址，然后紧接着

  if (base.charAt(0) !== '/') {    base = '/' + base;  }

当url第一个字符不是/的时候加上/，这里非常明显是一个BUG

我的是绝对地址http://static.com第一个字符当然不是/，
所以才会由之前的http://api.com/http:/static.com/test这样的网址

修改

if(/^([a-z]+:)?\/\//i.test(base)){}else if (base.charAt(0) !== '/') {  base = '/' + base;}

为了尽量少破坏源码，这里加了一个空的if，当url是由协议开始时，认为是绝对路径。
* 绝对路径还有一种形式是 //static.com

测试

经过第一次修改，再次访问页面依然有问题，访问的页面依然是http://api.com/http:/static.com/test

继续分析

再次跟踪源码后发现
vue-router.esm.js

//2006行~2016行  HTML5History.prototype.push = function push (location, onComplete, onAbort) {    var this$1 = this;    var ref = this;    var fromRoute = ref.current;    this.transitionTo(location, function (route) {      pushState(cleanPath(this$1.base + route.fullPath));      handleScroll(this$1.router, route, fromRoute, false);      onComplete && onComplete(route);    }, onAbort);  };//561行~563行function cleanPath (path) {  return path.replace(/\/\//g, '/')}

在发生pushState之前，他还会对url再次进行处理cleanPath
而这里的处理更简单，更粗暴，问题也更大。

他直接将2个斜杠//替换为1个斜杠/，话说如果连续3个斜杠怎么办？

所以在处理http://static.com/test地址的时候，其实会被处理成http:/static.com/test 又变成相对路径了...

继续修改

function cleanPath (path) {    var ishttp = /^([a-z]+:)?\/\//i.exec(path);    var http = Array.isArray(ishttp) ? ishttp[0] : '';    return http + path.substr(http.length).replace(/\/{2,}/g, '/');}

如果是协议开始，则排除协议内容之后，将2个或2个以上连续在一起的斜杠替换为1个斜杠。

** 完成提交pull
https://github.com/vuejs/vue-router/pull/1353/files
话说vue-router的url处理比起Url.js来说真的是太粗暴了...