Upgrade to Django 3.2.13

[ktohalloran]

Overview

Upgrades to Django 3.2.13 following the release of a point release to fix a security vulnerability.

Notes

When running test and runserver, the following deprecation warning appears in the console:

/usr/local/lib/python3.6/dist-packages/boto/plugin.py:40: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses

This seems to be an issue with dependency boto that has still not been addressed in their latest release.

While testing, I ran into some issues with my instance of OTP. As they're occurring on both develop and this branch, it seems unlikely to be caused by this upgrade; however, these errors mean my tests are failing so it would be worth testing this theory on another machine.

Testing Instructions

Note: These testing instructions have largely been copied from PR 1320.

• Provision your app VM to update Django vagrant provision app
• Next, run tests and the dev server to make sure that you don't see any deprecation warnings from Django (other than the ones mentioned above from boto):
  • vagrant ssh app
  • cd /opt/app/python/cac_tripplanner
  • python3 -Wall manage.py test
  • sudo systemctl stop cac-tripplanner-app (This stops the server process that comes up by default when you start the VM; you need to do this before starting up the Django dev server manually because you'll get a port conflict if you don't)
  • python3 -Wall manage.py runserver
• Last, click around the app and make sure everything works as it used to.

Checklist

• No gulp lint warnings
• No python lint warnings
• Python tests pass

Connects #1330

[ddohler]

Works great! I'm not sure what's up with the OTP test failures you observed; I'm not able to replicate them on my machine:

vagrant@app:/opt/app/python/cac_tripplanner$ python3 -Wall manage.py test
/usr/local/lib/python3.6/dist-packages/boto/plugin.py:40: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp
Creating test database for alias 'default'...
System check identified no issues (0 silenced).
.[2022-05-11 15:29:36 -0400] [4596] [WARNING] Bad Request: /map/reachable
......[2022-05-11 15:29:42 -0400] [4596] [WARNING] Not Found: /learn/unpublished-comm/
............................[2022-05-11 15:29:44 -0400] [4596] [WARNING] Method Not Allowed (GET): /link/shorten/
[2022-05-11 15:29:44 -0400] [4596] [WARNING] Method Not Allowed: /link/shorten/
[2022-05-11 15:29:44 -0400] [4596] [WARNING] Bad Request: /link/shorten/
[2022-05-11 15:29:44 -0400] [4596] [WARNING] Method Not Allowed (POST): /link/abcdefghijklmnopqrstuv
[2022-05-11 15:29:44 -0400] [4596] [WARNING] Method Not Allowed: /link/abcdefghijklmnopqrstuv
[2022-05-11 15:29:44 -0400] [4596] [WARNING] Not Found: /link/abcdefghijklmnopqrstuv
.
----------------------------------------------------------------------
Ran 36 tests in 8.570s

OK
Destroying test database for alias 'default'...

Could you open an issue for the OTP failures? My guess at the moment is that they may be related to different Virtualbox versions or something OS-specific. But I think we're good to go to merge this.

[ddohler]

Good catch! I don't know how I missed this. 🤦

[ktohalloran]

Thank you for reviewing! I'm glad the tests didn't fail on your machine, too. I've made #1332 to look into the failures later.