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 OAuth2 auth in PyGVFS #3758

Closed
xloya opened this issue Jun 4, 2024 · 3 comments · Fixed by #5030
Closed

[Improvement] Support OAuth2 auth in PyGVFS #3758

xloya opened this issue Jun 4, 2024 · 3 comments · Fixed by #5030
Assignees
Labels
0.7.0 Release v0.7.0 improvement Improvements on everything

Comments

@xloya
Copy link
Contributor

xloya commented Jun 4, 2024

What would you like to be improved?

Support OAuth2 authentication in PyGVFS so that it can interact with the Gravitino server. This is supported in Java GVFS and we should also support it in PyGVFS.

How should we improve?

No response

@xloya xloya added the improvement Improvements on everything label Jun 4, 2024
@jerryshao
Copy link
Contributor

It should already be supported, right? @xloya

@xloya
Copy link
Contributor Author

xloya commented Aug 1, 2024

It should already be supported, right? @xloya

Nope, we only support the Python client through Oauth2 authentication to access the server: #4011. Supporting for Oauth2 authentication in PyGVFS still requires some codes. See: /~https://github.com/apache/gravitino/blob/main/clients/client-python/gravitino/filesystem/gvfs.py#L116

@jerryshao
Copy link
Contributor

I see, thanks.

@xloya xloya self-assigned this Sep 27, 2024
@xloya xloya changed the title [Improvement] Support Oauth2 auth in PyGVFS [Improvement] Support OAuth2 auth in PyGVFS Oct 8, 2024
@jerryshao jerryshao added the 0.7.0 Release v0.7.0 label Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.7.0 Release v0.7.0 improvement Improvements on everything
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants