mirrors/dendrite
1
0
Fork 0
mirror of https://github.com/matrix-org/dendrite.git synced 2026-01-07 06:03:09 -06:00
Code Issues Packages Projects Releases Wiki Activity

Fix a lint warning.

Browse source
This commit is contained in:
Tommie Gannert 2022-05-27 23:08:53 +02:00
parent ffa527340a
commit b8844fb1e2
2 changed files with 3 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
clientapi/auth/sso/oidc.go
View file

@ -35,7 +35,7 @@ type oidcIdentityProvider struct {
mu sync.Mutex mu sync.Mutex
} }
func newOIDCIdentityProvider(cfg *config.IdentityProvider, hc *http.Client) (*oidcIdentityProvider, error) { func newOIDCIdentityProvider(cfg *config.IdentityProvider, hc *http.Client) *oidcIdentityProvider {
return &oidcIdentityProvider{ return &oidcIdentityProvider{
oauth2IdentityProvider: &oauth2IdentityProvider{ oauth2IdentityProvider: &oauth2IdentityProvider{
cfg: cfg, cfg: cfg,
@ -48,7 +48,7 @@ func newOIDCIdentityProvider(cfg *config.IdentityProvider, hc *http.Client) (*oi
displayNamePath: "name", displayNamePath: "name",
suggestedUserIDPath: "preferred_username", suggestedUserIDPath: "preferred_username",
}, },
}, nil }
} }
func (p *oidcIdentityProvider) AuthorizationURL(ctx context.Context, callbackURL, nonce string) (string, error) { func (p *oidcIdentityProvider) AuthorizationURL(ctx context.Context, callbackURL, nonce string) (string, error) {

6
clientapi/auth/sso/sso.go
View file

@ -46,11 +46,7 @@ func NewAuthenticator(ctx context.Context, cfg *config.SSO) (*Authenticator, err
switch pcfg.Type { switch pcfg.Type {
case config.SSOTypeOIDC: case config.SSOTypeOIDC:
p, err := newOIDCIdentityProvider(&pcfg, hc) a.providers[pcfg.ID] = newOIDCIdentityProvider(&pcfg, hc)
if err != nil {
return nil, fmt.Errorf("failed to create OpenID Connect provider %q: %w", pcfg.ID, err)
}
a.providers[pcfg.ID] = p
case config.SSOTypeGitHub: case config.SSOTypeGitHub:
a.providers[pcfg.ID] = newGitHubIdentityProvider(&pcfg, hc) a.providers[pcfg.ID] = newGitHubIdentityProvider(&pcfg, hc)
default: default: