From d02e7d9e6a03c266e161ac0b375be4ec32c61b58 Mon Sep 17 00:00:00 2001 From: Unknwon Date: Mon, 13 Feb 2017 04:20:03 -0500 Subject: [PATCH] install: no need to check SMTPFrom is not set (#4118) --- gogs.go | 2 +- routers/install.go | 14 ++++++++------ templates/.VERSION | 2 +- 3 files changed, 10 insertions(+), 8 deletions(-) diff --git a/gogs.go b/gogs.go index 5666e73ea..1db11bf3b 100644 --- a/gogs.go +++ b/gogs.go @@ -16,7 +16,7 @@ import ( "github.com/gogits/gogs/modules/setting" ) -const APP_VER = "0.9.145.0212" +const APP_VER = "0.9.145.0213" func init() { setting.AppVer = APP_VER diff --git a/routers/install.go b/routers/install.go index 1df875ae2..f46508323 100644 --- a/routers/install.go +++ b/routers/install.go @@ -249,12 +249,14 @@ func InstallPost(ctx *context.Context, form auth.InstallForm) { } // Make sure FROM field is valid - _, err := mail.ParseAddress(form.SMTPFrom) - if err != nil { - ctx.Data["Err_SMTP"] = true - ctx.Data["Err_SMTPFrom"] = true - ctx.RenderWithErr(ctx.Tr("install.invalid_smtp_from", err), INSTALL, &form) - return + if len(form.SMTPFrom) > 0 { + _, err := mail.ParseAddress(form.SMTPFrom) + if err != nil { + ctx.Data["Err_SMTP"] = true + ctx.Data["Err_SMTPFrom"] = true + ctx.RenderWithErr(ctx.Tr("install.invalid_smtp_from", err), INSTALL, &form) + return + } } // Check logic loophole between disable self-registration and no admin account. diff --git a/templates/.VERSION b/templates/.VERSION index d12102fdd..4adcf93eb 100644 --- a/templates/.VERSION +++ b/templates/.VERSION @@ -1 +1 @@ -0.9.145.0212 \ No newline at end of file +0.9.145.0213 \ No newline at end of file