Merge pull request #92 from snebel29/fix/proxy_root_domain_required_issue_39

Make PROXY_ROOT_DOMAIN required

Author: Shraya Ramani

internal/auth/options.go

@@ -157,6 +157,9 @@ func (o *Options) Validate() error {
 	if len(o.EmailDomains) == 0 {
 		msgs = append(msgs, "missing setting for email validation: email-domain required.\n      use email-domain=* to authorize all email addresses")
 	}
+	if len(o.ProxyRootDomains) == 0 {
+		msgs = append(msgs, "missing setting: proxy-root-domain")
+	}
 	if o.ProxyClientID == "" {
 		msgs = append(msgs, "missing setting: proxy-client-id")
 	}

internal/auth/options_test.go

@@ -18,6 +18,7 @@ func testOptions() *Options {
 	o.EmailDomains = []string{"*"}
 	o.ProxyClientID = "abcdef"
 	o.ProxyClientSecret = "testtest"
+	o.ProxyRootDomains = []string{"*"}
 	o.StatsdHost = "statsdhost"
 	o.StatsdPort = 12344
 	o.Host = "/"
@@ -43,6 +44,7 @@ func TestNewOptions(t *testing.T) {
 		"missing setting: cookie-secret",
 		"missing setting: client-id",
 		"missing setting: client-secret",
+		"missing setting: proxy-root-domain",
 		"missing setting: proxy-client-id",
 		"missing setting: proxy-client-secret",
 		"missing setting: required-host-header",