2017-03-11 14:30:29 +00:00
|
|
|
// Copyright 2017 The Gitea Authors. All rights reserved.
|
2022-11-27 18:20:29 +00:00
|
|
|
// SPDX-License-Identifier: MIT
|
2017-03-11 14:30:29 +00:00
|
|
|
|
2022-09-02 19:18:23 +00:00
|
|
|
package integration
|
2017-03-11 14:30:29 +00:00
|
|
|
|
|
|
|
import (
|
2020-11-14 16:53:43 +00:00
|
|
|
"fmt"
|
2017-04-25 07:24:51 +00:00
|
|
|
"net/http"
|
2020-11-14 16:53:43 +00:00
|
|
|
"strings"
|
2017-03-11 14:30:29 +00:00
|
|
|
"testing"
|
|
|
|
|
2021-11-16 08:53:21 +00:00
|
|
|
"code.gitea.io/gitea/models/unittest"
|
2021-11-24 09:49:20 +00:00
|
|
|
user_model "code.gitea.io/gitea/models/user"
|
2017-04-25 07:24:51 +00:00
|
|
|
"code.gitea.io/gitea/modules/setting"
|
[GITEA] Allow changing the email address before activation
During registration, one may be required to give their email address, to
be verified and activated later. However, if one makes a mistake, a
typo, they may end up with an account that cannot be activated due to
having a wrong email address.
They can still log in, but not change the email address, thus, no way to
activate it without help from an administrator.
To remedy this issue, lets allow changing the email address for logged
in, but not activated users.
This fixes gitea#17785.
Signed-off-by: Gergely Nagy <forgejo@gergo.csillger.hu>
(cherry picked from commit aaaece28e4c6a8980cef932e224e84933d7c9262)
(cherry picked from commit 639dafabec0a5c1f943b44ca02f72c5ba2fc5e10)
2023-12-07 11:57:16 +00:00
|
|
|
"code.gitea.io/gitea/modules/test"
|
2022-06-26 14:19:22 +00:00
|
|
|
"code.gitea.io/gitea/modules/translation"
|
2022-09-02 19:18:23 +00:00
|
|
|
"code.gitea.io/gitea/tests"
|
2021-11-17 12:34:35 +00:00
|
|
|
|
2020-11-14 16:53:43 +00:00
|
|
|
"github.com/stretchr/testify/assert"
|
2017-04-25 07:24:51 +00:00
|
|
|
)
|
2017-03-11 14:30:29 +00:00
|
|
|
|
|
|
|
func TestSignup(t *testing.T) {
|
2022-09-02 19:18:23 +00:00
|
|
|
defer tests.PrepareTestEnv(t)()
|
2017-04-28 13:23:28 +00:00
|
|
|
|
2017-04-25 07:24:51 +00:00
|
|
|
setting.Service.EnableCaptcha = false
|
|
|
|
|
2017-06-17 04:49:45 +00:00
|
|
|
req := NewRequestWithValues(t, "POST", "/user/sign_up", map[string]string{
|
|
|
|
"user_name": "exampleUser",
|
|
|
|
"email": "exampleUser@example.com",
|
2019-11-09 03:40:37 +00:00
|
|
|
"password": "examplePassword!1",
|
|
|
|
"retype": "examplePassword!1",
|
2017-06-17 04:49:45 +00:00
|
|
|
})
|
2022-03-23 04:54:07 +00:00
|
|
|
MakeRequest(t, req, http.StatusSeeOther)
|
2017-03-11 14:30:29 +00:00
|
|
|
|
2017-04-25 07:24:51 +00:00
|
|
|
// should be able to view new user's page
|
2017-06-10 00:41:36 +00:00
|
|
|
req = NewRequest(t, "GET", "/exampleUser")
|
2017-07-07 19:36:47 +00:00
|
|
|
MakeRequest(t, req, http.StatusOK)
|
2017-03-11 14:30:29 +00:00
|
|
|
}
|
2020-11-14 16:53:43 +00:00
|
|
|
|
2021-07-15 19:19:48 +00:00
|
|
|
func TestSignupAsRestricted(t *testing.T) {
|
2022-09-02 19:18:23 +00:00
|
|
|
defer tests.PrepareTestEnv(t)()
|
2021-07-15 19:19:48 +00:00
|
|
|
|
|
|
|
setting.Service.EnableCaptcha = false
|
|
|
|
setting.Service.DefaultUserIsRestricted = true
|
|
|
|
|
|
|
|
req := NewRequestWithValues(t, "POST", "/user/sign_up", map[string]string{
|
|
|
|
"user_name": "restrictedUser",
|
|
|
|
"email": "restrictedUser@example.com",
|
|
|
|
"password": "examplePassword!1",
|
|
|
|
"retype": "examplePassword!1",
|
|
|
|
})
|
2022-03-23 04:54:07 +00:00
|
|
|
MakeRequest(t, req, http.StatusSeeOther)
|
2021-07-15 19:19:48 +00:00
|
|
|
|
|
|
|
// should be able to view new user's page
|
|
|
|
req = NewRequest(t, "GET", "/restrictedUser")
|
|
|
|
MakeRequest(t, req, http.StatusOK)
|
|
|
|
|
2022-08-16 02:22:25 +00:00
|
|
|
user2 := unittest.AssertExistsAndLoadBean(t, &user_model.User{Name: "restrictedUser"})
|
2021-07-15 19:19:48 +00:00
|
|
|
assert.True(t, user2.IsRestricted)
|
|
|
|
}
|
|
|
|
|
2020-11-14 16:53:43 +00:00
|
|
|
func TestSignupEmail(t *testing.T) {
|
2022-09-02 19:18:23 +00:00
|
|
|
defer tests.PrepareTestEnv(t)()
|
2020-11-14 16:53:43 +00:00
|
|
|
|
|
|
|
setting.Service.EnableCaptcha = false
|
|
|
|
|
|
|
|
tests := []struct {
|
|
|
|
email string
|
|
|
|
wantStatus int
|
|
|
|
wantMsg string
|
|
|
|
}{
|
2022-06-26 14:19:22 +00:00
|
|
|
{"exampleUser@example.com\r\n", http.StatusOK, translation.NewLocale("en-US").Tr("form.email_invalid")},
|
|
|
|
{"exampleUser@example.com\r", http.StatusOK, translation.NewLocale("en-US").Tr("form.email_invalid")},
|
|
|
|
{"exampleUser@example.com\n", http.StatusOK, translation.NewLocale("en-US").Tr("form.email_invalid")},
|
2022-03-23 04:54:07 +00:00
|
|
|
{"exampleUser@example.com", http.StatusSeeOther, ""},
|
2020-11-14 16:53:43 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
for i, test := range tests {
|
|
|
|
req := NewRequestWithValues(t, "POST", "/user/sign_up", map[string]string{
|
|
|
|
"user_name": fmt.Sprintf("exampleUser%d", i),
|
|
|
|
"email": test.email,
|
|
|
|
"password": "examplePassword!1",
|
|
|
|
"retype": "examplePassword!1",
|
|
|
|
})
|
|
|
|
resp := MakeRequest(t, req, test.wantStatus)
|
|
|
|
if test.wantMsg != "" {
|
|
|
|
htmlDoc := NewHTMLParser(t, resp.Body)
|
|
|
|
assert.Equal(t,
|
|
|
|
test.wantMsg,
|
|
|
|
strings.TrimSpace(htmlDoc.doc.Find(".ui.message").Text()),
|
|
|
|
)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
[GITEA] Allow changing the email address before activation
During registration, one may be required to give their email address, to
be verified and activated later. However, if one makes a mistake, a
typo, they may end up with an account that cannot be activated due to
having a wrong email address.
They can still log in, but not change the email address, thus, no way to
activate it without help from an administrator.
To remedy this issue, lets allow changing the email address for logged
in, but not activated users.
This fixes gitea#17785.
Signed-off-by: Gergely Nagy <forgejo@gergo.csillger.hu>
(cherry picked from commit aaaece28e4c6a8980cef932e224e84933d7c9262)
(cherry picked from commit 639dafabec0a5c1f943b44ca02f72c5ba2fc5e10)
2023-12-07 11:57:16 +00:00
|
|
|
|
|
|
|
func TestSignupEmailChangeForInactiveUser(t *testing.T) {
|
|
|
|
defer tests.PrepareTestEnv(t)()
|
|
|
|
|
|
|
|
// Disable the captcha & enable email confirmation for registrations
|
|
|
|
defer test.MockVariableValue(&setting.Service.EnableCaptcha, false)()
|
|
|
|
defer test.MockVariableValue(&setting.Service.RegisterEmailConfirm, true)()
|
|
|
|
|
|
|
|
// Create user
|
|
|
|
req := NewRequestWithValues(t, "POST", "/user/sign_up", map[string]string{
|
|
|
|
"user_name": "exampleUserX",
|
|
|
|
"email": "wrong-email@example.com",
|
|
|
|
"password": "examplePassword!1",
|
|
|
|
"retype": "examplePassword!1",
|
|
|
|
})
|
|
|
|
MakeRequest(t, req, http.StatusOK)
|
|
|
|
|
|
|
|
session := loginUserWithPassword(t, "exampleUserX", "examplePassword!1")
|
|
|
|
|
|
|
|
// Verify that the initial e-mail is the wrong one.
|
|
|
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{Name: "exampleUserX"})
|
|
|
|
assert.Equal(t, "wrong-email@example.com", user.Email)
|
|
|
|
|
|
|
|
// Change the email address
|
|
|
|
req = NewRequestWithValues(t, "POST", "/user/activate", map[string]string{
|
|
|
|
"email": "fine-email@example.com",
|
|
|
|
})
|
|
|
|
session.MakeRequest(t, req, http.StatusSeeOther)
|
|
|
|
|
|
|
|
// Verify that the email was updated
|
|
|
|
user = unittest.AssertExistsAndLoadBean(t, &user_model.User{Name: "exampleUserX"})
|
|
|
|
assert.Equal(t, "fine-email@example.com", user.Email)
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestSignupEmailChangeForActiveUser(t *testing.T) {
|
|
|
|
defer tests.PrepareTestEnv(t)()
|
|
|
|
|
|
|
|
// Disable the captcha & enable email confirmation for registrations
|
|
|
|
defer test.MockVariableValue(&setting.Service.EnableCaptcha, false)()
|
|
|
|
defer test.MockVariableValue(&setting.Service.RegisterEmailConfirm, false)()
|
|
|
|
|
|
|
|
// Create user
|
|
|
|
req := NewRequestWithValues(t, "POST", "/user/sign_up", map[string]string{
|
|
|
|
"user_name": "exampleUserY",
|
|
|
|
"email": "wrong-email-2@example.com",
|
|
|
|
"password": "examplePassword!1",
|
|
|
|
"retype": "examplePassword!1",
|
|
|
|
})
|
|
|
|
MakeRequest(t, req, http.StatusSeeOther)
|
|
|
|
|
|
|
|
session := loginUserWithPassword(t, "exampleUserY", "examplePassword!1")
|
|
|
|
|
|
|
|
// Verify that the initial e-mail is the wrong one.
|
|
|
|
user := unittest.AssertExistsAndLoadBean(t, &user_model.User{Name: "exampleUserY"})
|
|
|
|
assert.Equal(t, "wrong-email-2@example.com", user.Email)
|
|
|
|
|
|
|
|
// Changing the email for a validated address is not available
|
|
|
|
req = NewRequestWithValues(t, "POST", "/user/activate", map[string]string{
|
|
|
|
"email": "fine-email-2@example.com",
|
|
|
|
})
|
|
|
|
session.MakeRequest(t, req, http.StatusNotFound)
|
|
|
|
|
|
|
|
// Verify that the email remained unchanged
|
|
|
|
user = unittest.AssertExistsAndLoadBean(t, &user_model.User{Name: "exampleUserY"})
|
|
|
|
assert.Equal(t, "wrong-email-2@example.com", user.Email)
|
|
|
|
}
|