kodev-org
/
vikunja
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(openid): manually fetch providers 

Partially reverts fcdcdcf46a
Resolves https://github.com/go-vikunja/vikunja/issues/1165
This commit is contained in:
kolaente 2025-07-28 11:39:57 +02:00
parent 84a9428b4e
commit de917467cb
No known key found for this signature in database
GPG Key ID: F40E70337AB24C9B
1 changed files with 7 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
pkg/modules/auth/openid/providers.go
View File

@ -89,32 +89,21 @@ func GetAllProviders() (providers []*Provider, err error) {
// GetProvider retrieves a provider from keyvalue
func GetProvider(key string) (provider *Provider, err error) {
result, err := keyvalue.Remember("openid_provider_"+key, func() (any, error) {
_, err := GetAllProviders() // This will put all providers in cache
provider = &Provider{}
exists, err := keyvalue.GetWithValue("openid_provider_"+key, provider)
if err != nil {
return nil, err
}
if !exists {
_, err = GetAllProviders() // This will put all providers in cache
if err != nil {
return nil, err
}
provider := &Provider{}
_, err = keyvalue.GetWithValue("openid_provider_"+key, provider)
if err != nil {
return nil, err
}
return provider, nil
})
if err != nil {
return nil, err
}
// Handle type assertion safely - the memory backend strips pointer info when storing
switch v := result.(type) {
case *Provider:
provider = v
case Provider:
provider = &v
default:
log.Debugf("stored cached provider has an invalid type %T, value: %v", result, result)
return nil, fmt.Errorf("invalid cached provider type: %T", result)
}
err = provider.setOicdProvider()