关于vue-router路径计算问题
冰麟轻武 人气:0昨天刚刚发表了一个前端跨域新方案尝试,今天在开发中就遇到的了问题。
起因
前端使用的是vue-router组件的history
模式,但是由于我们的整个页面都是从static(静态资源站)load过来的,所以其他页面自然也需要跨域去拿,然而就在跨域的时候 vue-router 出了问题。
分析问题
我们的api站点在 api.com
而静态资源在 static.com,页面的base标签也指向static
<base href="http://static.com" rel="external nofollow" />
然而,在访问 test
模板时却跳到了http://api.com/http:/static.com/test
经过一些简单的断点调试,锁定了以下代码
[vue-router.esm.js][source]
//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][source]
//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来说真的是太粗暴了...
加载全部内容