meutel/oauth2_proxy
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Remove check for >0 upstreams

Browse Source 
When used solely for auth_request there is no upstream.
Instead of forcing users to set a dummy upstream, remove
the check.
This commit is contained in:
Christian Svensson 2017-07-20 13:35:06 +02:00
parent 3c51c914ac
commit 0b117133b9
2 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
options.go
View File

@ -121,9 +121,6 @@ func parseURL(to_parse string, urltype string, msgs []string) (*url.URL, []strin
func (o *Options) Validate() error { func (o *Options) Validate() error {
msgs := make([]string, 0) msgs := make([]string, 0)
if len(o.Upstreams) < 1 {
msgs = append(msgs, "missing setting: upstream")
}
if o.CookieSecret == "" { if o.CookieSecret == "" {
msgs = append(msgs, "missing setting: cookie-secret") msgs = append(msgs, "missing setting: cookie-secret")
} }

1
options_test.go
View File

@ -35,7 +35,6 @@ func TestNewOptions(t *testing.T) {
assert.NotEqual(t, nil, err) assert.NotEqual(t, nil, err)
expected := errorMsg([]string{ expected := errorMsg([]string{
"missing setting: upstream",
"missing setting: cookie-secret", "missing setting: cookie-secret",
"missing setting: client-id", "missing setting: client-id",
"missing setting: client-secret"}) "missing setting: client-secret"})