dd46428 · typeable/oauth2-proxy

git.iko.soy/typeable/oauth2-proxy

on master help sign in

Ignore spaces in authorized emails list

dd46428Stan Hu11 years ago

validator.go+2−1

@@ -53,7 +53,8 @@ func (um *UserMap) LoadAuthenticatedEmailsFile() {

53 }

54 updated := make(map[string]bool)

55 for _, r := range records {

56− updated[strings.ToLower(r[0])] = true

56+ address := strings.ToLower(strings.TrimSpace(r[0]))

57+ updated[address] = true

57 }

58 }

58 atomic.StorePointer(&um.m, unsafe.Pointer(&updated))

59 atomic.StorePointer(&um.m, unsafe.Pointer(&updated))

59}

60}

validator_test.go+13

@@ -147,3 +147,16 @@ func TestValidatorComparisonsAreCaseInsensitive(t *testing.T) {

147 t.Error("validated domains are not lower-cased")

148 }

149}

150+

151+func TestValidatorIgnoreSpacesInAuthEmails(t *testing.T) {

152+ vt := NewValidatorTest(t)

153+ defer vt.TearDown()

154+

155+ vt.WriteEmails(t, []string{" foo.bar@example.com "})

156+ domains := []string(nil)

157+ validator := vt.NewValidator(domains, nil)

158+

159+ if !validator("foo.bar@example.com") {

160+ t.Error("email should validate")

161+ }

162+}