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

[Improvement] Support Oauth2AuthProvider for Python client #3755

Closed
xloya opened this issue Jun 4, 2024 · 2 comments · Fixed by #4011
Closed

[Improvement] Support Oauth2AuthProvider for Python client #3755

xloya opened this issue Jun 4, 2024 · 2 comments · Fixed by #4011
Assignees
Labels
improvement Improvements on everything

Comments

@xloya
Copy link
Contributor

xloya commented Jun 4, 2024

What would you like to be improved?

Currently the Python client does not support Oauth2 authentication, which is supported in the Java client. We should support this capability in the Python client.

How should we improve?

No response

@xloya xloya added the improvement Improvements on everything label Jun 4, 2024
@xloya xloya changed the title [Improvement] Support Oauth2Provider for Python client [Improvement] Support Oauth2AuthProvider for Python client Jun 4, 2024
@noidname01
Copy link
Collaborator

@xloya Hi, I would like to work on this.

@xloya
Copy link
Contributor Author

xloya commented Jun 26, 2024

@noidname01 Sure, go ahead.

jerryshao pushed a commit that referenced this issue Jul 19, 2024
…ython client (#4011)

### What changes were proposed in this pull request?

* Add `OAuth2TokenProvider` and `DefaultOAuth2TokenProvider` in
`client-python`
* There are some components and tests missing because it would be a big
code change if they were also done in this PR, they will be added in the
following PRs
	- [ ] Error Handling: #4173 
	- [ ] Integration Test: #4208 
* Modify test file structure, and found issue #4136, solve it by reset
environment variable.

### Why are the changes needed?

Fix: #3755, #4136

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

Add UT and tested by `./gradlew clients:client-python:unittest`

---------

Co-authored-by: TimWang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvements on everything
Projects
None yet
3 participants