-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Fixing cgi.escape DeprecationWarning #23939
Fixing cgi.escape DeprecationWarning #23939
Conversation
Thanks for the pull request, @leomartinez2019! I've created OSPR-4490 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams. Feel free to add as much of the following information to the ticket:
All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here. |
Hi @natabene, could you trigger the tests? |
jenkins run all |
@leomartinez2019 Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future. |
23aeffe
to
b293d8e
Compare
jenkins run all |
jenkins run quality |
@leomartinez2019 Please address the failing quality test. |
Jenkins reports these quality warnings:
Here the solution is suggested: Which consists of using the function HTML() from: openedx.core.djangolib.markup With the suggested changes this test passes:
|
b293d8e
to
b54e35d
Compare
jenkins run all |
Unfortunately when this test passes: This other fails: And viceversa, so I am still researching to find a solution |
@leomartinez2019 I would say that |
It's not that simple. The assertion is a real failure.
In this case, the XML actually contains |
jenkins run all |
dddc56b
to
8cea844
Compare
Hi @natabene, could you trigger them again? |
jenkins run all |
jenkins run py38 python |
jenkins run all |
jenkins run py38 python |
1 similar comment
jenkins run py38 python |
- Change cgi.escape to html.escape - Add quote=False to html.escape - Use function HTML() to address python-wrap-html warning
8cea844
to
943bd6f
Compare
jenkins run all |
jenkins run py38 python |
Your PR has finished running tests. The following contexts failed:
|
@leomartinez2019 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
EdX Release Notice: This PR has been deployed to the production environment. |
This pull request fixes the DeprecationWarning as stated in:
https://build.testeng.edx.org/job/edx-platform-python-pipeline-master/warning_5freport_5fall_2ehtml/
cgi.escape is deprecated in Python3:
https://docs.python.org/3.5/library/cgi.html#cgi.escape
Use html.escape instead:
https://docs.python.org/3.5/library/html.html#html.escape
@felipemontoya
@morenol
@Squirrel18
FYI: @jmbowman