Merge pull request #2802 from davidswu/2.7-autoredirect
[2.7] default autoredirect to false
This commit is contained in:
commit
e702d95cfd
1 changed files with 7 additions and 4 deletions
|
@ -162,11 +162,14 @@ func checkOptions(options map[string]interface{}) (tokenAccessOptions, error) {
|
||||||
|
|
||||||
opts.realm, opts.issuer, opts.service, opts.rootCertBundle = vals[0], vals[1], vals[2], vals[3]
|
opts.realm, opts.issuer, opts.service, opts.rootCertBundle = vals[0], vals[1], vals[2], vals[3]
|
||||||
|
|
||||||
autoRedirect, ok := options["autoredirect"].(bool)
|
autoRedirectVal, ok := options["autoredirect"]
|
||||||
if !ok {
|
if ok {
|
||||||
return opts, fmt.Errorf("token auth requires a valid option bool: autoredirect")
|
autoRedirect, ok := autoRedirectVal.(bool)
|
||||||
|
if !ok {
|
||||||
|
return opts, fmt.Errorf("token auth requires a valid option bool: autoredirect")
|
||||||
|
}
|
||||||
|
opts.autoRedirect = autoRedirect
|
||||||
}
|
}
|
||||||
opts.autoRedirect = autoRedirect
|
|
||||||
|
|
||||||
return opts, nil
|
return opts, nil
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue