chore(deps): bump requests from 2.28.2 to 2.31.0

[dependabot]

Bumps requests from 2.28.2 to 2.31.0.

Release notes

Sourced from requests's releases.

v2.31.0

2.31.0 (2023-05-22)

Security

• Versions of Requests between v2.3.0 and v2.30.0 are vulnerable to potential forwarding of Proxy-Authorization headers to destination servers when following HTTPS redirects.

  When proxies are defined with user info (https://user:pass@proxy:8080), Requests will construct a Proxy-Authorization header that is attached to the request to authenticate with the proxy.

  In cases where Requests receives a redirect response, it previously reattached the Proxy-Authorization header incorrectly, resulting in the value being sent through the tunneled connection to the destination server. Users who rely on defining their proxy credentials in the URL are strongly encouraged to upgrade to Requests 2.31.0+ to prevent unintentional leakage and rotate their proxy credentials once the change has been fully deployed.

  Users who do not use a proxy or do not supply their proxy credentials through the user information portion of their proxy URL are not subject to this vulnerability.

  Full details can be read in our Github Security Advisory and CVE-2023-32681.

v2.30.0

2.30.0 (2023-05-03)

Dependencies

• ⚠️ Added support for urllib3 2.0. ⚠️

  This may contain minor breaking changes so we advise careful testing and reviewing https://urllib3.readthedocs.io/en/latest/v2-migration-guide.html prior to upgrading.

  Users who wish to stay on urllib3 1.x can pin to urllib3<2.

v2.29.0

2.29.0 (2023-04-26)

Improvements

• Requests now defers chunked requests to the urllib3 implementation to improve standardization. (#6226)
• Requests relaxes header component requirements to support bytes/str subclasses. (#6356)

Changelog

Sourced from requests's changelog.

2.31.0 (2023-05-22)

Security

• Versions of Requests between v2.3.0 and v2.30.0 are vulnerable to potential forwarding of Proxy-Authorization headers to destination servers when following HTTPS redirects.

  When proxies are defined with user info (https://user:pass@proxy:8080), Requests will construct a Proxy-Authorization header that is attached to the request to authenticate with the proxy.

  In cases where Requests receives a redirect response, it previously reattached the Proxy-Authorization header incorrectly, resulting in the value being sent through the tunneled connection to the destination server. Users who rely on defining their proxy credentials in the URL are strongly encouraged to upgrade to Requests 2.31.0+ to prevent unintentional leakage and rotate their proxy credentials once the change has been fully deployed.

  Users who do not use a proxy or do not supply their proxy credentials through the user information portion of their proxy URL are not subject to this vulnerability.

  Full details can be read in our Github Security Advisory and CVE-2023-32681.

2.30.0 (2023-05-03)

Dependencies

• ⚠️ Added support for urllib3 2.0. ⚠️

  This may contain minor breaking changes so we advise careful testing and reviewing https://urllib3.readthedocs.io/en/latest/v2-migration-guide.html prior to upgrading.

  Users who wish to stay on urllib3 1.x can pin to urllib3<2.

2.29.0 (2023-04-26)

Improvements

• Requests now defers chunked requests to the urllib3 implementation to improve standardization. (#6226)
• Requests relaxes header component requirements to support bytes/str subclasses. (#6356)

Commits

• 147c851 v2.31.0
• 74ea7cf Merge pull request from GHSA-j8r2-6x86-q33q
• 3022253 test on pypy 3.8 and pypy 3.9 on windows and macos (#6424)
• b639e66 test on py3.12 (#6448)
• d3d5044 Fixed a small typo (#6452)
• 2ad18e0 v2.30.0
• f2629e9 Remove strict parameter (#6434)
• 87d63de v2.29.0
• 51716c4 enable the warnings plugin (#6416)
• a7da1ab try on ubuntu 22.04 (#6418)
• Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  You can disable automated security fix PRs for this repo from the Security Alerts page.

[thekaveman]

Install updates

Outside the devcontainer

Checkout this branch and rebuild the devcontainer

Inside the devcontainer

Confirm package updates (edited for brevity / relevant packages)

$ pip freeze
...
requests==2.31.0
...
urllib3==2.0.3
...

Run pytest

✅ All pytests passed locally

Inside the devcontainer

$ export PYTHONWARNINGS="default"
$ pytest
==================================================================== test session starts
platform linux -- Python 3.9.17, pytest-7.3.2, pluggy-1.0.0
rootdir: /home/calitp/src
plugins: mock-3.11.1, cov-4.1.0
collected 17 items
...
===================================================================== 17 passed in 0.42s

Additional manual tests

✅ Confirmed Agency Card verification (local benefits, local eligibility-server)

Configure server

Outside the devcontainer.

Replace the reference to eligibility-api in requirements.txt to point to this PR's commit:

git+https://github.com/cal-itp/eligibility-api.git@18c08eda233aa65b050409f0897277fc8b94b85d#egg=eligibility_api

Rebuild the image:

$ bin/build.sh

Configure Benefits

Outside the devcontainer.

See the note on this related PR review: cal-itp/eligibility-server#259 (review)

Ensure Benefits compose.yml points to the newly built eligibility_server:latest image

server:
  image: eligibility_server:latest

Update the reference to eligibility-api in pyprojecy.toml using the PEP508 format:

dependencies = [
  ...
  "eligibility_api @ git+https://github.com/cal-itp/eligibility-api.git@18c08eda233aa65b050409f0897277fc8b94b85d#egg=eligibility_api",
  ...
]

Rebuild the benefits image:

$ docker compose build client

Start the client and server:

$ docker compose up -d client server

Attach terminals to both running client and server to confirm they are using the updated package:

Confirm eligibility verification.

urllib3 2.0 considerations

See the note about urllib3 2.0 on this related PR review: cal-itp/benefits#1406 (review)

For this repository, requests is listed (this PR) and so is responses, which is at the latest version as of this review and is only used for testing/mocks:

$ pipdeptree --reverse | grep "requires: urllib3"
├── requests==2.31.0 [requires: urllib3>=1.21.1,<3]
└── responses==0.23.1 [requires: urllib3>=1.25.10]