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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Issue 3220 Update Django version for CVEs #3238

Merged

Conversation

changhsinlee
Copy link
Contributor

@changhsinlee changhsinlee commented Nov 10, 2022

What is this?

Fixes issue #3220 by bumping to Django==3.2.16 that fixed two CVEs related to Django.

I followed what was done here when it was last updated to 3.2.14

Dev notes

I only have a Mac so I tested locally with sqlite and one of the tests (see below) failed for the develop base branch when I pulled from the most recent -- yet I don't see a failed build on GitHub 馃. All other tests passed.

Therefor I'm not sure if my test setup was correct. I tried following both README to install and then copied how GItHub workflow script setup the testing env to no avail. This may not be an issue though.

FAILED tests/test_api.py::test_creating_activating_new_ml_backend[201-http://my.super.ai-4] - assert 5 == 4

@changhsinlee
Copy link
Contributor Author

Huh looks like it failed because the Github action checkout is broken for forks?
WeblateOrg/weblate#6240 (comment)

I can open another one without forking if that works

@wesleylima wesleylima merged commit a4d7f3b into HumanSignal:develop Nov 15, 2022
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.

None yet

2 participants