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

fix(html_tag): encode urls in meta tag #159

Merged
merged 2 commits into from Dec 18, 2019
Merged

Conversation

@curbengh
Copy link
Contributor

curbengh commented Dec 18, 2019

While working on hexojs/hexo#3983, I found url in open graph or twitter card tag is html-escaped instead of url-encoded.

@coveralls

This comment has been minimized.

Copy link

coveralls commented Dec 18, 2019

Coverage Status

Coverage increased (+0.01%) to 96.462% when pulling d4bf265 on curbengh:html-tag-og into 621c9e3 on hexojs:master.

@curbengh curbengh requested a review from SukkaW Dec 18, 2019
@curbengh curbengh mentioned this pull request Dec 18, 2019
0 of 2 tasks complete
@SukkaW
SukkaW approved these changes Dec 18, 2019
@curbengh curbengh merged commit 41ea824 into hexojs:master Dec 18, 2019
3 checks passed
3 checks passed
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
coverage/coveralls Coverage increased (+0.01%) to 96.462%
Details
@curbengh curbengh deleted the curbengh:html-tag-og branch Dec 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.