Merge pull request #59 from spinnaker/more-mail-properties

use spring boot mail starter instead of java mail directly.
spinnaker · Jan 7, 2016 · 6f83668 · 6f83668
2 parents e3a139c + 8bcdafc
commit 6f83668
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 42 deletions.
diff --git a/echo-notifications/echo-notifications.gradle b/echo-notifications/echo-notifications.gradle
@@ -23,7 +23,7 @@ dependencies {
     compile spinnaker.dependency("kork")
     compile spinnaker.dependency("rxJava")
     compile "javax.mail:mail:1.4.1"
-    compile "org.springframework:spring-context-support:${spinnaker.version('spring')}"
+    compile "org.springframework.boot:spring-boot-starter-mail:${spinnaker.version('springBoot')}"
     compile "org.apache.velocity:velocity:1.7"
     compile "org.jsoup:jsoup:1.8.2"
     testCompile "com.icegreen:greenmail:1.4.0"

diff --git a/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/config/EmailConfig.groovy b/echo-notifications/src/main/groovy/com/netflix/spinnaker/echo/config/EmailConfig.groovy
diff --git a/...ications/src/main/groovy/com/netflix/spinnaker/echo/email/EmailNotificationService.groovy b/...ications/src/main/groovy/com/netflix/spinnaker/echo/email/EmailNotificationService.groovy
@@ -65,7 +65,7 @@ class EmailNotificationService implements NotificationService {
     MimeMessage message = javaMailSender.createMimeMessage()
     MimeMessageHelper helper = new MimeMessageHelper(message)
 
-    helper.setBcc(to)
+    helper.setTo(to)
     helper.setFrom(from)
     helper.setText(text)
     helper.setSubject(subject)