Arsharth/dex
1
0
Fork 0
forked from mystiq/dex
Code Issues Pull requests Projects Releases Packages Wiki Activity

api: don't create a user if you can't send them an email

Browse source
This commit is contained in:
Joe Bowers 2015-09-25 15:08:45 -07:00
parent e8f347a738
commit f115015a3f
2 changed files with 34 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
user/api/api.go
View file

@ -132,6 +132,16 @@ func (u *UsersAPI) CreateUser(creds Creds, usr schema.User, redirURL url.URL) (s
return schema.UserCreateResponse{}, mapError(err) return schema.UserCreateResponse{}, mapError(err)
} }
metadata, err := u.clientIdentityRepo.Metadata(creds.ClientID)
if err != nil {
return schema.UserCreateResponse{}, mapError(err)
}
validRedirURL, err := client.ValidRedirectURL(&redirURL, metadata.RedirectURLs)
if err != nil {
return schema.UserCreateResponse{}, ErrorInvalidRedirectURL
}
id, err := u.manager.CreateUser(schemaUserToUser(usr), user.Password(hash), u.localConnectorID) id, err := u.manager.CreateUser(schemaUserToUser(usr), user.Password(hash), u.localConnectorID)
if err != nil { if err != nil {
return schema.UserCreateResponse{}, mapError(err) return schema.UserCreateResponse{}, mapError(err)
@ -144,16 +154,6 @@ func (u *UsersAPI) CreateUser(creds Creds, usr schema.User, redirURL url.URL) (s
usr = userToSchemaUser(userUser) usr = userToSchemaUser(userUser)
metadata, err := u.clientIdentityRepo.Metadata(creds.ClientID)
if err != nil {
return schema.UserCreateResponse{}, mapError(err)
}
validRedirURL, err := client.ValidRedirectURL(&redirURL, metadata.RedirectURLs)
if err != nil {
return schema.UserCreateResponse{}, ErrorInvalidRedirectURL
}
url, err := u.emailer.SendResetPasswordEmail(usr.Email, validRedirURL, creds.ClientID) url, err := u.emailer.SendResetPasswordEmail(usr.Email, validRedirURL, creds.ClientID)
// An email is sent only if we don't get a link and there's no error. // An email is sent only if we don't get a link and there's no error.

30
user/api/api_test.go
View file

@ -238,7 +238,7 @@ func TestCreateUser(t *testing.T) {
{ {
creds: goodCreds, creds: goodCreds,
usr: schema.User{ usr: schema.User{
Email: "newuser@example.com", Email: "newuser01@example.com",
DisplayName: "New User", DisplayName: "New User",
EmailVerified: true, EmailVerified: true,
Admin: false, Admin: false,
@ -248,7 +248,7 @@ func TestCreateUser(t *testing.T) {
wantResponse: schema.UserCreateResponse{ wantResponse: schema.UserCreateResponse{
EmailSent: true, EmailSent: true,
User: &schema.User{ User: &schema.User{
Email: "newuser@example.com", Email: "newuser01@example.com",
DisplayName: "New User", DisplayName: "New User",
EmailVerified: true, EmailVerified: true,
Admin: false, Admin: false,
@ -259,7 +259,7 @@ func TestCreateUser(t *testing.T) {
{ {
creds: goodCreds, creds: goodCreds,
usr: schema.User{ usr: schema.User{
Email: "newuser@example.com", Email: "newuser02@example.com",
DisplayName: "New User", DisplayName: "New User",
EmailVerified: true, EmailVerified: true,
Admin: false, Admin: false,
@ -269,7 +269,7 @@ func TestCreateUser(t *testing.T) {
wantResponse: schema.UserCreateResponse{ wantResponse: schema.UserCreateResponse{
User: &schema.User{ User: &schema.User{
Email: "newuser@example.com", Email: "newuser02@example.com",
DisplayName: "New User", DisplayName: "New User",
EmailVerified: true, EmailVerified: true,
Admin: false, Admin: false,
@ -281,7 +281,7 @@ func TestCreateUser(t *testing.T) {
{ {
creds: goodCreds, creds: goodCreds,
usr: schema.User{ usr: schema.User{
Email: "newuser@example.com", Email: "newuser03@example.com",
DisplayName: "New User", DisplayName: "New User",
EmailVerified: true, EmailVerified: true,
Admin: false, Admin: false,
@ -293,7 +293,7 @@ func TestCreateUser(t *testing.T) {
{ {
creds: badCreds, creds: badCreds,
usr: schema.User{ usr: schema.User{
Email: "newuser@example.com", Email: "newuser04@example.com",
DisplayName: "New User", DisplayName: "New User",
EmailVerified: true, EmailVerified: true,
Admin: false, Admin: false,
@ -313,6 +313,24 @@ func TestCreateUser(t *testing.T) {
if err != tt.wantErr { if err != tt.wantErr {
t.Errorf("case %d: want=%q, got=%q", i, tt.wantErr, err) t.Errorf("case %d: want=%q, got=%q", i, tt.wantErr, err)
} }
tok := ""
for {
list, tok, err := api.ListUsers(goodCreds, 100, tok)
if err != nil {
t.Fatalf("case %d: unexpected error: %v", i, err)
break
}
for _, u := range list {
if u.Email == tt.usr.Email {
t.Errorf("case %d: got an error but user was still created", i)
}
}
if tok == "" {
break
}
}
continue continue
} }
if err != nil { if err != nil {