diff --git a/models/login_source.go b/models/login_source.go index fd977e20a..57b1d56bb 100644 --- a/models/login_source.go +++ b/models/login_source.go @@ -21,6 +21,7 @@ import ( "code.gitea.io/gitea/modules/setting" "code.gitea.io/gitea/modules/timeutil" "code.gitea.io/gitea/modules/util" + gouuid "github.com/google/uuid" jsoniter "github.com/json-iterator/go" "xorm.io/xorm" @@ -116,6 +117,7 @@ func (cfg *SMTPConfig) ToDB() ([]byte, error) { // PAMConfig holds configuration for the PAM login source. type PAMConfig struct { ServiceName string // pam service (e.g. system-auth) + EmailDomain string } // FromDB fills up a PAMConfig from serialized format. @@ -696,15 +698,26 @@ func LoginViaPAM(user *User, login, password string, sourceID int64, cfg *PAMCon // Allow PAM sources with `@` in their name, like from Active Directory username := pamLogin + email := pamLogin idx := strings.Index(pamLogin, "@") if idx > -1 { username = pamLogin[:idx] } + if ValidateEmail(email) != nil { + if cfg.EmailDomain != "" { + email = fmt.Sprintf("%s@%s", username, cfg.EmailDomain) + } else { + email = fmt.Sprintf("%s@%s", username, setting.Service.NoReplyAddress) + } + if ValidateEmail(email) != nil { + email = gouuid.New().String() + "@localhost" + } + } user = &User{ LowerName: strings.ToLower(username), Name: username, - Email: pamLogin, + Email: email, Passwd: password, LoginType: LoginPAM, LoginSource: sourceID, diff --git a/modules/forms/auth_form.go b/modules/forms/auth_form.go index 7cf6b9fcd..30621cadf 100644 --- a/modules/forms/auth_form.go +++ b/modules/forms/auth_form.go @@ -51,6 +51,7 @@ type AuthenticationForm struct { TLS bool SkipVerify bool PAMServiceName string + PAMEmailDomain string Oauth2Provider string Oauth2Key string Oauth2Secret string diff --git a/options/locale/locale_en-US.ini b/options/locale/locale_en-US.ini index e679e1e87..a2320a20e 100644 --- a/options/locale/locale_en-US.ini +++ b/options/locale/locale_en-US.ini @@ -2281,6 +2281,7 @@ auths.allowed_domains_helper = Leave empty to allow all domains. Separate multip auths.enable_tls = Enable TLS Encryption auths.skip_tls_verify = Skip TLS Verify auths.pam_service_name = PAM Service Name +auths.pam_email_domain = PAM Email Domain (optional) auths.oauth2_provider = OAuth2 Provider auths.oauth2_icon_url = Icon URL auths.oauth2_clientID = Client ID (Key) diff --git a/routers/admin/auths.go b/routers/admin/auths.go index 12d0a2ccf..f96c810d3 100644 --- a/routers/admin/auths.go +++ b/routers/admin/auths.go @@ -239,6 +239,7 @@ func NewAuthSourcePost(ctx *context.Context) { case models.LoginPAM: config = &models.PAMConfig{ ServiceName: form.PAMServiceName, + EmailDomain: form.PAMEmailDomain, } case models.LoginOAuth2: config = parseOAuth2Config(form) @@ -346,6 +347,7 @@ func EditAuthSourcePost(ctx *context.Context) { case models.LoginPAM: config = &models.PAMConfig{ ServiceName: form.PAMServiceName, + EmailDomain: form.PAMEmailDomain, } case models.LoginOAuth2: config = parseOAuth2Config(form) diff --git a/templates/admin/auth/edit.tmpl b/templates/admin/auth/edit.tmpl index 6b2485839..96da7d2ed 100644 --- a/templates/admin/auth/edit.tmpl +++ b/templates/admin/auth/edit.tmpl @@ -188,6 +188,10 @@ +
+ + +
{{end}} diff --git a/templates/admin/auth/new.tmpl b/templates/admin/auth/new.tmpl index c252f7d3f..7742e131d 100644 --- a/templates/admin/auth/new.tmpl +++ b/templates/admin/auth/new.tmpl @@ -38,6 +38,8 @@
+ +