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

chore: Pin requests to avoid issues with urllib3 2.0 #50

Merged
merged 1 commit into from
May 18, 2023

Conversation

guptaa3
Copy link
Contributor

@guptaa3 guptaa3 commented May 17, 2023

Pinning version of requests to avoid urllib3 error witnessed in urllib 3.0 added in requests 2.30.0 version

Error observed:
ImportError: urllib3 v2.0 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with OpenSSL 1.0.2k-fips 26 Jan 2017. See: urllib3/urllib3#2168

Cause Identified:
https://urllib3.readthedocs.io/en/latest/v2-migration-guide.html
https://github.com/psf/requests/releases/tag/v2.30.0
https://github.com/psf/requests/blob/2ad18e0e10e7d7ecd5384c378f25ec8821a10a29/setup.py#L64

Pinning version of requests to avoid urllib3 error witnessed in urllib 3.0 added in requests 2.30.0 version 

ImportError: urllib3 v2.0 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with OpenSSL 1.0.2k-fips 26 Jan 2017. See: urllib3/urllib3#2168
@guptaa3
Copy link
Contributor Author

guptaa3 commented May 17, 2023

@eryanRM request your review please - Thank you!

@pnadolny13 pnadolny13 changed the title Update pyproject.toml chore: Update pyproject.toml May 17, 2023
@edgarrmondragon
Copy link
Member

@pnadolny13 @guptaa3 I think bumping singer-sdk to >=0.27.0 should fix issues seen with urllib3 2.0. See meltano/sdk#1680.

@guptaa3
Copy link
Contributor Author

guptaa3 commented May 17, 2023

Thank you @edgarrmondragon but I think the tap should work irrespective of the singer-sdk version used, what do you think ?

@edgarrmondragon
Copy link
Member

Thank you @edgarrmondragon but I think the tap should work irrespective of the singer-sdk version used, what do you think ?

@guptaa3 Agreed 👍. Just commenting on how this will become a non-issue after the singer-sdk dependency is bumped 🙂.

@edgarrmondragon edgarrmondragon changed the title chore: Update pyproject.toml chore: Pin requests to avoid issues with urllib3 2.0 May 17, 2023
@guptaa3
Copy link
Contributor Author

guptaa3 commented May 18, 2023

@kgpayne @edgarrmondragon Thank you for approving the PR :)
Request you to merge the PR please as I do not have write access
Thank you!

@kgpayne kgpayne merged commit 756c27b into MeltanoLabs:main May 18, 2023
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.

3 participants