Skip to content

docs: fix unescaped html tag #802

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

Merged
merged 2 commits into from
Apr 21, 2020
Merged

Conversation

Billy4195
Copy link
Contributor

If the description of the method contains any HTML tag, it will break the HTML rendering.
This PR escapes the HTML tag to prevent this problem depending on the different runtime python version

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 16, 2020
@busunkim96 busunkim96 changed the title docs: Fix unescaped html tag docs: fix unescaped html tag Jan 16, 2020
@busunkim96 busunkim96 added the kokoro:run Add this label to force Kokoro to re-run the tests. label Jan 16, 2020
@kokoro-team kokoro-team removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Jan 16, 2020
If the description of the method contains any HTML tag, it will
break the HTML rendering.
This commit escapes the html tag to prevent this problem.
* Use html standard lib when using python3
* Use cgi standard lib when using python2

Refs googleapis#791
@busunkim96 busunkim96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 21, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 21, 2020
@busunkim96 busunkim96 merged commit 45cace8 into googleapis:master Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants