-
Notifications
You must be signed in to change notification settings - Fork 15
Open
Labels
bugSomething isn't workingSomething isn't working
Description
See https://github.com/coupergateway/couper-examples/actions/workflows/test.yaml
This results from oidcConfig.GetIssuer() (creating a sync request for the OIDC configuration) now (after #796) being in the path for couper verify:
- Verify.Execute()
- runtime.NewServerConfiguration()
- configureAccessControls()
- oauth2.NewOidcClient()
- oidcConfig.GetIssuer()
Another minor issue:
issuer, err := oidcConfig.GetIssuer()
if err != nil {
return nil, errors.Oauth2.With(err)
^^^^^^^^^^^^^^^^^^^^^^^If we keep the JWT parser being created in NewOidcClient(), err should not be wrapped here, as errors.Oauth2 is for runtime errors.
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working