mirror of
https://github.com/fatedier/frp.git
synced 2026-05-15 08:05:49 -06:00
[PR #3723] [CLOSED] Feature/retry access token #4817
Labels
No labels
In Progress
WIP
WaitingForInfo
bug
doc
duplicate
easy
enhancement
future
help wanted
invalid
lifecycle/stale
need-issue-template
need-usage-help
no plan
proposal
pull-request
question
todo
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/frp#4817
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/fatedier/frp/pull/3723
Author: @RobKenis
Created: 10/24/2023
Status: ❌ Closed
Base:
dev← Head:feature/retry-access-token📝 Commits (2)
96c5dbdRetry retrieving an access token with exponential backoffd0e5852Make maxRetries and delay configurable with config files📊 Changes
3 files changed (+42 additions, -5 deletions)
View changed files
📝
pkg/auth/oidc.go(+27 -5)📝
pkg/config/v1/client.go(+5 -0)📝
pkg/config/v1/common.go(+10 -0)📄 Description
Summary
🤖 Generated by Copilot at
d0e5852This pull request adds retry support for OIDC authentication in frp clients. It defines a new
AuthRetryConfigstruct to store the retry parameters, and uses it in theOidcAuthProvider,ClientCommonConfig, andAuthOIDCClientConfigstructs. It also modifies thepkg/auth/oidc.goandpkg/config/v1/client.gofiles to implement the retry logic and configuration parsing.WHY
When the OIDC provider is unavailable when a new token is requested, the client
is stuck for an indefinite state. This is only resolved by restarting the client or
the server.
With this feature, we want to achieve a scenario where a token is requested for a
set amount of retries when the OIDC provider is not available.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.