Pass provider options to Cidaas provider
Co-authored-by: Jan Larwig <jan@larwig.com>
This commit is contained in:
parent
de787a2948
commit
652d16692d
|
|
@ -33,17 +33,16 @@ const (
|
||||||
)
|
)
|
||||||
|
|
||||||
// NewCIDAASProvider initiates a new CIDAASProvider
|
// NewCIDAASProvider initiates a new CIDAASProvider
|
||||||
func NewCIDAASProvider(p *ProviderData) *CIDAASProvider {
|
func NewCIDAASProvider(p *ProviderData, opts options.Provider) *CIDAASProvider {
|
||||||
p.setProviderDefaults(providerDefaults{
|
p.setProviderDefaults(providerDefaults{
|
||||||
name: CidaasProviderName,
|
name: CidaasProviderName,
|
||||||
scope: CidaasDefaultScope,
|
scope: CidaasDefaultScope,
|
||||||
})
|
})
|
||||||
|
|
||||||
|
opts.InsecureSkipNonce = true // Should this be configurable or not?! Do you need to skip the nonce for Cidaas?
|
||||||
|
|
||||||
return &CIDAASProvider{
|
return &CIDAASProvider{
|
||||||
OIDCProvider: &OIDCProvider{
|
OIDCProvider: NewOIDCProvider(p, opts.OIDCConfig),
|
||||||
ProviderData: p,
|
|
||||||
SkipNonce: true,
|
|
||||||
},
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -116,6 +115,7 @@ func (p *CIDAASProvider) extractGroups(respJSON *simplejson.Json) ([]string, err
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
||||||
var groupsClaimList GroupsClaimList
|
var groupsClaimList GroupsClaimList
|
||||||
err = json.Unmarshal(rawGroupsClaim, &groupsClaimList)
|
err = json.Unmarshal(rawGroupsClaim, &groupsClaimList)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
|
|
||||||
|
|
@ -46,7 +46,7 @@ func NewProvider(providerConfig options.Provider) (Provider, error) {
|
||||||
case options.BitbucketProvider:
|
case options.BitbucketProvider:
|
||||||
return NewBitbucketProvider(providerData, providerConfig.BitbucketConfig), nil
|
return NewBitbucketProvider(providerData, providerConfig.BitbucketConfig), nil
|
||||||
case options.CidaasProvider:
|
case options.CidaasProvider:
|
||||||
return NewCIDAASProvider(providerData), nil
|
return NewCIDAASProvider(providerData, providerConfig), nil
|
||||||
case options.DigitalOceanProvider:
|
case options.DigitalOceanProvider:
|
||||||
return NewDigitalOceanProvider(providerData), nil
|
return NewDigitalOceanProvider(providerData), nil
|
||||||
case options.FacebookProvider:
|
case options.FacebookProvider:
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue