Skip to content

Commit

Permalink
Fix sender of issue notifications
Browse files Browse the repository at this point in the history
It is the FROM field in mailer configuration that needs be used,
not the USER field, which is for authentication.

Closes gogs#3615
  • Loading branch information
strk committed Sep 7, 2016
1 parent 491407d commit e85e850
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion models/mail.go
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@ func composeIssueMessage(issue *Issue, doer *User, tplName base.TplName, tos []s
if err != nil {
log.Error(3, "HTMLString (%s): %v", tplName, err)
}
msg := mailer.NewMessageFrom(tos, fmt.Sprintf(`"%s" <%s>`, doer.DisplayName(), setting.MailService.User), subject, content)
msg := mailer.NewMessageFrom(tos, fmt.Sprintf(`"%s" <%s>`, doer.DisplayName(), setting.MailService.From), subject, content)
msg.Info = fmt.Sprintf("Subject: %s, %s", subject, info)
return msg
}
Expand Down

0 comments on commit e85e850

Please sign in to comment.