Skip to content

feat: add refresh feature for OAuth2 refresh token - #581

Closed
anencore94 wants to merge 0 commit into
google:mainfrom
anencore94:fix/oauth2_refresh
Closed

anencore94 wants to merge 0 commit into
google:mainfrom
anencore94:fix/oauth2_refresh

Conversation

@anencore94

@anencore94 anencore94 commented May 6, 2025 •

Copy link
Copy Markdown
Contributor

fixes "# TODO(cheliu): Implement token refresh flow" @pwwpche

  • add expiry field for OAuth2Auth
  • use refresh_token if there isn't any access_token or expired.

@hangfei

hangfei commented May 30, 2025

Copy link
Copy Markdown
Collaborator

@seanzhou1023 Please help review this change.

@anencore94 please resolve the conflict.s

@hangfei
hangfei requested a review from seanzhou1023 May 30, 2025 19:18
@hangfei hangfei added the tools [Component] This issue is related to tools label May 30, 2025
@anencore94 anencore94 closed this May 31, 2025
@anencore94
anencore94 force-pushed the fix/oauth2_refresh branch from d197c83 to 7fdc6b4 Compare May 31, 2025 08:05
@anencore94

Copy link
Copy Markdown
Contributor Author

I removed my original branch by mistake, so I submit another pull request with same change list. PTAL @hangfei @seanzhou1023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tools [Component] This issue is related to tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants