From db6b71ad03c40e9b01f40bc986a915a8b4bd03b2 Mon Sep 17 00:00:00 2001 From: Unknown Date: Thu, 15 May 2014 12:08:53 -0400 Subject: [PATCH 1/2] Update issue-user when change milestone --- models/issue.go | 45 ++++++++++++++++++++++++++++++++++++++++--- routers/repo/issue.go | 2 +- 2 files changed, 43 insertions(+), 4 deletions(-) diff --git a/models/issue.go b/models/issue.go index 7dd69267ae3..d3eb89c356e 100644 --- a/models/issue.go +++ b/models/issue.go @@ -161,6 +161,13 @@ func GetIssueCountByPoster(uid, rid int64, isClosed bool) int64 { return count } +// .___ ____ ___ +// | | ______ ________ __ ____ | | \______ ___________ +// | |/ ___// ___/ | \_/ __ \| | / ___// __ \_ __ \ +// | |\___ \ \___ \| | /\ ___/| | /\___ \\ ___/| | \/ +// |___/____ >____ >____/ \___ >______//____ >\___ >__| +// \/ \/ \/ \/ \/ + // IssueUser represents an issue-user relation. type IssueUser struct { Id int64 @@ -404,6 +411,13 @@ type Label struct { NumOpenIssues int `xorm:"-"` } +// _____ .__.__ __ +// / \ |__| | ____ _______/ |_ ____ ____ ____ +// / \ / \| | | _/ __ \ / ___/\ __\/ _ \ / \_/ __ \ +// / Y \ | |_\ ___/ \___ \ | | ( <_> ) | \ ___/ +// \____|__ /__|____/\___ >____ > |__| \____/|___| /\___ > +// \/ \/ \/ \/ \/ + // Milestone represents a milestone of repository. type Milestone struct { Id int64 @@ -517,7 +531,7 @@ func ChangeMilestoneStatus(m *Milestone, isClosed bool) (err error) { } // ChangeMilestoneAssign changes assignment of milestone for issue. -func ChangeMilestoneAssign(oldMid, mid int64, isIssueClosed bool) (err error) { +func ChangeMilestoneAssign(oldMid, mid int64, issue *Issue) (err error) { sess := orm.NewSession() defer sess.Close() if err = sess.Begin(); err != nil { @@ -531,7 +545,7 @@ func ChangeMilestoneAssign(oldMid, mid int64, isIssueClosed bool) (err error) { } m.NumIssues-- - if isIssueClosed { + if issue.IsClosed { m.NumClosedIssues-- } if m.NumIssues > 0 { @@ -543,6 +557,12 @@ func ChangeMilestoneAssign(oldMid, mid int64, isIssueClosed bool) (err error) { sess.Rollback() return err } + + rawSql := "UPDATE `issue_user` SET milestone_id = 0 WHERE issue_id = ?" + if _, err = sess.Exec(rawSql, issue.Id); err != nil { + sess.Rollback() + return err + } } if mid > 0 { @@ -551,7 +571,7 @@ func ChangeMilestoneAssign(oldMid, mid int64, isIssueClosed bool) (err error) { return err } m.NumIssues++ - if isIssueClosed { + if issue.IsClosed { m.NumClosedIssues++ } m.Completeness = m.NumClosedIssues * 100 / m.NumIssues @@ -559,6 +579,12 @@ func ChangeMilestoneAssign(oldMid, mid int64, isIssueClosed bool) (err error) { sess.Rollback() return err } + + rawSql := "UPDATE `issue_user` SET milestone_id = ? WHERE issue_id = ?" + if _, err = sess.Exec(rawSql, m.Id, issue.Id); err != nil { + sess.Rollback() + return err + } } return sess.Commit() } @@ -587,9 +613,22 @@ func DeleteMilestone(m *Milestone) (err error) { sess.Rollback() return err } + + rawSql = "UPDATE `issue_user` SET milestone_id = 0 WHERE milestone_id = ?" + if _, err = sess.Exec(rawSql, m.Id); err != nil { + sess.Rollback() + return err + } return sess.Commit() } +// _________ __ +// \_ ___ \ ____ _____ _____ ____ _____/ |_ +// / \ \/ / _ \ / \ / \_/ __ \ / \ __\ +// \ \___( <_> ) Y Y \ Y Y \ ___/| | \ | +// \______ /\____/|__|_| /__|_| /\___ >___| /__| +// \/ \/ \/ \/ \/ + // Issue types. const ( IT_PLAIN = iota // Pure comment. diff --git a/routers/repo/issue.go b/routers/repo/issue.go index db0eff9cde2..83b84e289a8 100644 --- a/routers/repo/issue.go +++ b/routers/repo/issue.go @@ -426,7 +426,7 @@ func UpdateIssueMilestone(ctx *middleware.Context) { // Not check for invalid milestone id and give responsibility to owners. issue.MilestoneId = mid - if err = models.ChangeMilestoneAssign(oldMid, mid, issue.IsClosed); err != nil { + if err = models.ChangeMilestoneAssign(oldMid, mid, issue); err != nil { ctx.Handle(500, "issue.UpdateIssueMilestone(ChangeMilestoneAssign)", err) return } else if err = models.UpdateIssue(issue); err != nil { From 4744996f9a68ef0de8bc87b3c22c0bde9a305847 Mon Sep 17 00:00:00 2001 From: Unknown Date: Thu, 15 May 2014 14:46:04 -0400 Subject: [PATCH 2/2] Make gmail auth work --- models/login.go | 37 +++++++++++++++++++-------------- models/user.go | 2 +- modules/mailer/mailer.go | 8 +++---- routers/admin/auths.go | 2 +- templates/admin/auths.tmpl | 2 +- templates/admin/auths/edit.tmpl | 18 +++++++++------- templates/admin/auths/new.tmpl | 16 +++++++++++--- templates/admin/users/edit.tmpl | 2 +- 8 files changed, 52 insertions(+), 35 deletions(-) diff --git a/models/login.go b/models/login.go index aa82eb0036c..005a1f5b766 100644 --- a/models/login.go +++ b/models/login.go @@ -5,6 +5,7 @@ package models import ( + "crypto/tls" "encoding/json" "errors" "fmt" @@ -133,7 +134,7 @@ func AddSource(source *LoginSource) error { } func UpdateSource(source *LoginSource) error { - _, err := orm.AllCols().Id(source.Id).Update(source) + _, err := orm.Id(source.Id).AllCols().Update(source) return err } @@ -197,7 +198,7 @@ func LoginUser(uname, passwd string) (*User, error) { if err == nil { return u, nil } else { - log.Warn("try ldap login", source.Name, "by", uname, "error:", err) + log.Warn("Fail to login(%s) by LDAP(%s): %v", uname, source.Name, err) } } else if source.Type == LT_SMTP { u, err := LoginUserSMTPSource(nil, uname, passwd, @@ -205,7 +206,7 @@ func LoginUser(uname, passwd string) (*User, error) { if err == nil { return u, nil } else { - log.Warn("try smtp login", source.Name, "by", uname, "error:", err) + log.Warn("Fail to login(%s) by SMTP(%s): %v", uname, source.Name, err) } } } @@ -217,12 +218,9 @@ func LoginUser(uname, passwd string) (*User, error) { hasSource, err := orm.Id(u.LoginSource).Get(&source) if err != nil { return nil, err - } - if !hasSource { + } else if !hasSource { return nil, ErrLoginSourceNotExist - } - - if !source.IsActived { + } else if !source.IsActived { return nil, ErrLoginSourceNotActived } @@ -296,20 +294,25 @@ var ( SMTPAuths = []string{SMTP_PLAIN, SMTP_LOGIN} ) -func SmtpAuth(addr string, a smtp.Auth, tls bool) error { - c, err := smtp.Dial(addr) +func SmtpAuth(host string, port int, a smtp.Auth, useTls bool) error { + c, err := smtp.Dial(fmt.Sprintf("%s:%d", host, port)) if err != nil { return err } defer c.Close() - if tls { + if err = c.Hello("gogs"); err != nil { + return err + } + + if useTls { if ok, _ := c.Extension("STARTTLS"); ok { - if err = c.StartTLS(nil); err != nil { + config := &tls.Config{ServerName: host} + if err = c.StartTLS(config); err != nil { return err } } else { - return errors.New("smtp server unsupported tls") + return errors.New("SMTP server unsupported TLS") } } @@ -333,11 +336,13 @@ func LoginUserSMTPSource(user *User, name, passwd string, sourceId int64, cfg *S } else if cfg.Auth == SMTP_LOGIN { auth = LoginAuth(name, passwd) } else { - return nil, errors.New("Unsupported smtp auth type") + return nil, errors.New("Unsupported SMTP auth type") } - err := SmtpAuth(fmt.Sprintf("%s:%d", cfg.Host, cfg.Port), auth, cfg.TLS) - if err != nil { + if err := SmtpAuth(cfg.Host, cfg.Port, auth, cfg.TLS); err != nil { + if strings.Contains(err.Error(), "Username and Password not accepted") { + return nil, ErrUserNotExist + } return nil, err } diff --git a/models/user.go b/models/user.go index c5c474ee6f2..dd0498501f9 100644 --- a/models/user.go +++ b/models/user.go @@ -34,7 +34,7 @@ var ( ErrUserNameIllegal = errors.New("User name contains illegal characters") ErrLoginSourceNotExist = errors.New("Login source does not exist") ErrLoginSourceNotActived = errors.New("Login source is not actived") - ErrUnsupportedLoginType = errors.New("Login source is unknow") + ErrUnsupportedLoginType = errors.New("Login source is unknown") ) // User represents the object of individual and member of organization. diff --git a/modules/mailer/mailer.go b/modules/mailer/mailer.go index 63861d870ec..a293beb15bb 100644 --- a/modules/mailer/mailer.go +++ b/modules/mailer/mailer.go @@ -72,16 +72,14 @@ func Send(msg *Message) (int, error) { // get message body content := msg.Content() - auth := smtp.PlainAuth("", base.MailService.User, base.MailService.Passwd, host[0]) - if len(msg.To) == 0 { return 0, fmt.Errorf("empty receive emails") - } - - if len(msg.Body) == 0 { + } else if len(msg.Body) == 0 { return 0, fmt.Errorf("empty email body") } + auth := smtp.PlainAuth("", base.MailService.User, base.MailService.Passwd, host[0]) + if msg.Massive { // send mail to multiple emails one by one num := 0 diff --git a/routers/admin/auths.go b/routers/admin/auths.go index bf4bade17bb..c4702afc81e 100644 --- a/routers/admin/auths.go +++ b/routers/admin/auths.go @@ -158,7 +158,7 @@ func EditAuthSourcePost(ctx *middleware.Context, form auth.AuthenticationForm) { } log.Trace("%s Authentication changed by admin(%s): %s", ctx.Req.RequestURI, - ctx.User.LowerName, strings.ToLower(form.AuthName)) + ctx.User.LowerName, form.AuthName) ctx.Redirect("/admin/auths") } diff --git a/templates/admin/auths.tmpl b/templates/admin/auths.tmpl index 87baeafd626..a0f7ba97908 100644 --- a/templates/admin/auths.tmpl +++ b/templates/admin/auths.tmpl @@ -19,7 +19,7 @@ Actived Updated Created - Operation + Edit diff --git a/templates/admin/auths/edit.tmpl b/templates/admin/auths/edit.tmpl index f2ba68fd277..5747c1ab516 100644 --- a/templates/admin/auths/edit.tmpl +++ b/templates/admin/auths/edit.tmpl @@ -115,16 +115,20 @@ - - {{end}}
+ {{if eq $type 3}} +
+
+ +
+
+ {{end}} +
- +
@@ -148,6 +148,16 @@
+
+
+ Tips +
+ +
+
GMail Setting:
+

Host: smtp.gmail.com, Post: 587, Enable TLS Encryption: true

+
+