Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

yilia_plus_full_url 对 https未处理 #47

Closed
csliubo opened this issue Mar 12, 2020 · 1 comment
Closed

yilia_plus_full_url 对 https未处理 #47

csliubo opened this issue Mar 12, 2020 · 1 comment

Comments

@csliubo
Copy link

csliubo commented Mar 12, 2020

❓问题

.config.yml 里 url 的schema如果是https,yilia_plus_full_url 生成的结果是错的。
假如url里配置的是: https://aaa.github.io
最后生成的是: https:/aaa.github.io/

具体原因是 scripts/helpers/yilia-plus-vendors.js 里的这段代码,在最后替换的时候并未将https考虑进去

  let fullUrl = (blogUrl + url).replace(/(\\|\/){2,}/g, '/')
    .replace(/((ht|f)tp\:(\\|\/)+)/ig, 'http://');

🤔结果 & 预期

预期生成结果是: https://aaa.github.io/

@JoeyBling
Copy link
Owner

感谢反馈,新版本已解决!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants