Skip to content

Commit

Permalink
fix(notification/slack): fix missing logger (#644)
Browse files Browse the repository at this point in the history
was causing slack notifications to repeat endlessly as the notification would be
sent, but the log statement afterwords would fail and return an error back to
orca which would retry.
  • Loading branch information
cfieber authored Sep 6, 2019
1 parent 4372e7b commit 32ef1e9
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,13 @@ import com.netflix.spinnaker.echo.api.Notification
import com.netflix.spinnaker.echo.controller.EchoResponse
import com.netflix.spinnaker.echo.notification.NotificationService
import com.netflix.spinnaker.echo.notification.NotificationTemplateEngine
import groovy.util.logging.Slf4j
import org.springframework.beans.factory.annotation.Autowired
import org.springframework.beans.factory.annotation.Value
import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty
import org.springframework.stereotype.Component

@Slf4j
@Component
@ConditionalOnProperty('slack.enabled')
class SlackNotificationService implements NotificationService {
Expand Down

0 comments on commit 32ef1e9

Please sign in to comment.