habsi/authentik
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

internal: Use loop instead of recursion in NewAPIController (#10745)

Browse Source 
use loop instead of recursion
This commit is contained in:
RMT
2024-08-16 21:04:07 +08:00
committed by GitHub
parent c3d3646645
commit d75cdfeaf1
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
internal/outpost/ak/api.go
View File

@ -77,11 +77,17 @@ func NewAPIController(akURL url.URL, token string) *APIController {
// Because we don't know the outpost UUID, we simply do a list and pick the first // Because we don't know the outpost UUID, we simply do a list and pick the first
// The service account this token belongs to should only have access to a single outpost // The service account this token belongs to should only have access to a single outpost
outposts, _, err := apiClient.OutpostsApi.OutpostsInstancesList(context.Background()).Execute() var outposts *api.PaginatedOutpostList
if err != nil { var err error
for {
outposts, _, err = apiClient.OutpostsApi.OutpostsInstancesList(context.Background()).Execute()
if err == nil {
break
}
log.WithError(err).Error("Failed to fetch outpost configuration, retrying in 3 seconds") log.WithError(err).Error("Failed to fetch outpost configuration, retrying in 3 seconds")
time.Sleep(time.Second * 3) time.Sleep(time.Second * 3)
return NewAPIController(akURL, token)
} }
if len(outposts.Results) < 1 { if len(outposts.Results) < 1 {
panic("No outposts found with given token, ensure the given token corresponds to an authenitk Outpost") panic("No outposts found with given token, ensure the given token corresponds to an authenitk Outpost")