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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump requests to 2.32.3 #7057

Merged
merged 1 commit into from
Jul 15, 2024
Merged

Bump requests to 2.32.3 #7057

merged 1 commit into from
Jul 15, 2024

Conversation

justinclift
Copy link
Member

What type of PR is this?

  • Other

Description

We recently updated the requests library to 2.32.2. There's a slightly newer point release available though, so we might as well move up to that.

How is this tested?

  • Unit tests (pytest, jest)
  • E2E Tests (Cypress)

@justinclift
Copy link
Member Author

@eradman @lucydodo @wlach @gaecoli Anyone up for approving this?

It's a straight forward version bump and passed the tests on my local system before creating the PR.

Copy link
Member

@lucydodo lucydodo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 😄

@lucydodo lucydodo merged commit 77f108d into master Jul 15, 2024
14 checks passed
@lucydodo lucydodo deleted the requests_2.32.3_v1 branch July 15, 2024 00:39
@justinclift
Copy link
Member Author

Thanks. 😄

@lucydodo
Copy link
Member

This is an already merged PR, but for reference, I'm also attaching the changes I identified during the review phase.

https://github.com/psf/requests/blob/main/HISTORY.md#2323-2024-05-29

2.32.3 (2024-05-29)
Bugfixes

Fixed bug breaking the ability to specify custom SSLContexts in sub-classes of HTTPAdapter. (#6716)
Fixed issue where Requests started failing to run on Python versions compiled without the ssl module. (#6724)

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.

2 participants