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

Update build.sh for Issue 330 #331

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

brianok-aws
Copy link

Pinning requests to 2.29.0 to avoid importing urllib3 2.0 and resolve the conflict with lack of support for that library in botocore to resolve Issue #330.

Issue #, if available: 330

Description of changes:
Running as a library referenced in Glue 4 results in the exception:
from neptune_python_utils.glue_neptune_connection_info import GlueNeptuneConnectionInfo
File "/tmp/neptune_python_utils.zip/neptune_python_utils/glue_neptune_connection_info.py",
...
line 34, in import botocore.httpsession File "/home/spark/.local/lib/python3.10/site-packages/botocore/httpsession.py", line 21, in from urllib3.util.ssl_ import ( ImportError: cannot import name 'DEFAULT_CIPHERS' from 'urllib3.util.ssl_' (/tmp/neptune_python_utils.zip/urllib3/util/ssl_.py)

This is due to a conflict between botocore not supporting urllib3 2.0 and newer versions of releases using urllib3 2.0. By pinning releases to 2.29.0, it resolves this conflict. This was tested by running the old library in Glue 4 and failing then running the revised library in Glue 4 and completing successfully.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pinning requests to 2.29.0 to avoid importing urllib3 2.0 and resolve the conflict with lack of support for that library in botocore to resolve Issues awslabs#330.
@Andrew8xx8
Copy link

I have the same issue. The fix works.

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