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

Upgrade google auth version #5

Merged

Conversation

deadlycoconuts
Copy link
Contributor

Context

This PR pins the version of the google-auth package in the SDK to >=2.17.3 as it is the first release which fixes the version of its urllib3 dependency at below version 2.0, because it is no longer compatible with the functions/objects defined in the google-auth package.

Copy link
Collaborator

@krithika369 krithika369 left a comment

Choose a reason for hiding this comment

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

LGTM!

@deadlycoconuts deadlycoconuts merged commit e23f097 into caraml-dev:main Jun 13, 2023
deadlycoconuts added a commit to caraml-dev/merlin that referenced this pull request Jun 13, 2023
**What this PR does / why we need it**:
This PR pins the version of the `caraml-auth-google` package in the SDK
to `0.0.0post6`, which fixes any potential dependency conflicts between
`urllib3>=2.0` and `google-auth`. See
caraml-dev/caraml-sdk#5 for more details.

**Which issue(s) this PR fixes**:
- errors using functions/objects defined in the `google.auth` package
when `urllib3>=2.0` is installed

Fixes #

**Does this PR introduce a user-facing change?**:
```release-note
NONE
```

**Checklist**

- [x] Added unit test, integration, and/or e2e tests
- [x] Tested locally
- [ ] Updated documentation
- [ ] Update Swagger spec if the PR introduce API changes
- [ ] Regenerated Golang and Python client if the PR introduce API
changes
@krithika369
Copy link
Collaborator

@deadlycoconuts Digging this PR up from its grave. The dependency should be google-auth>2.17.3 or google-auth>=2.18.0 and not >=2.17.3, right?

See: https://github.com/googleapis/google-auth-library-python/blob/v2.17.3/setup.py
vs https://github.com/googleapis/google-auth-library-python/blob/v2.18.0/setup.py#L32

@deadlycoconuts
Copy link
Contributor Author

Omg yes, did someone bump into this issue?

@krithika369
Copy link
Collaborator

Omg yes, did someone bump into this issue?

Not exactly. But can you help correct this here and in Merlin / Turing SDK too? Since there are newer google-auth SDK versions out there, the >=2.17.3 will resolve to the latest and is not a problem. But this will affect someone who happens to have precisely google-auth==2.17.3 in their environment.

@deadlycoconuts deadlycoconuts deleted the bump_google_auth_version branch September 7, 2023 09:18
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