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

feature(renderer): htmlEscape should escape quotes #647

Merged
merged 1 commit into from
Jun 26, 2020

Conversation

gdamore
Copy link
Collaborator

@gdamore gdamore commented Jun 25, 2020

This makes all single and double quotes escaped in SGML content,
improving the safety of using these characters inline.

Fixes #644

This makes all single and double quotes escaped in SGML content,
improving the safety of using these characters inline.
@gdamore
Copy link
Collaborator Author

gdamore commented Jun 25, 2020

This forced a lot of tests to be updated.

@codecov
Copy link

codecov bot commented Jun 25, 2020

Codecov Report

Merging #647 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #647   +/-   ##
=======================================
  Coverage   86.12%   86.12%           
=======================================
  Files          70       70           
  Lines        4685     4685           
=======================================
  Hits         4035     4035           
  Misses        400      400           
  Partials      250      250           

@xcoulon
Copy link
Member

xcoulon commented Jun 26, 2020

great, thanks for taking care of this @gdamore!

@xcoulon xcoulon merged commit 991725d into bytesparadise:master Jun 26, 2020
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

Successfully merging this pull request may close these issues.

htmlEscaper should escape quotes
2 participants