Merge pull request #889 from threatsim/feature/slack_service_changes

Adds in formatting for Slack
errbit · May 22, 2015 · 4f26840 · 4f26840
2 parents abbad62 + f16e413
commit 4f26840
Show file tree

Hide file tree

Showing 2 changed files with 82 additions and 2 deletions.
diff --git a/app/models/notification_services/slack_service.rb b/app/models/notification_services/slack_service.rb
@@ -18,7 +18,47 @@ def message_for_slack(problem)
   end
 
   def post_payload(problem)
-    {:text => message_for_slack(problem) }.to_json
+    {
+      :attachments => [
+        {
+           :fallback => message_for_slack(problem),
+           :pretext => "<#{problem_url(problem)}|Errbit - #{problem.app.name}: #{problem.error_class}>",
+           :color => "#D00000",
+           :fields => [
+              {
+                 :title => "Environment",
+                 :value => problem.environment,
+                 :short => false
+              },
+              {
+                 :title => "Location",
+                 :value => problem.where,
+                 :short => false
+              },
+              {
+                 :title => "Message",
+                 :value => problem.message.to_s,
+                 :short => false
+              },
+              {
+                 :title => "First Noticed",
+                 :value => problem.first_notice_at,
+                 :short => false
+              },
+              {
+                 :title => "Last Noticed",
+                 :value => problem.last_notice_at,
+                 :short => false
+              },
+              {
+                 :title => "Times Occurred",
+                 :value => problem.notices_count,
+                 :short => false
+              }
+           ]
+        }
+      ]
+    }.to_json
   end
 
   def create_notification(problem)

diff --git a/spec/models/notification_service/slack_service_spec.rb b/spec/models/notification_service/slack_service_spec.rb
@@ -6,7 +6,47 @@
     problem = notice.problem
 
     # faraday stubbing
-    payload = {:text => notification_service.message_for_slack(problem)}.to_json
+    payload = {
+      :attachments => [
+        {
+           :fallback => notification_service.message_for_slack(problem),
+           :pretext => "<#{notification_service.problem_url(problem)}|Errbit - #{problem.app.name}: #{problem.error_class}>",
+           :color => "#D00000",
+           :fields => [
+              {
+                 :title => "Environment",
+                 :value => problem.environment,
+                 :short => false
+              },
+              {
+                 :title => "Location",
+                 :value => problem.where,
+                 :short => false
+              },
+              {
+                 :title => "Message",
+                 :value => problem.message.to_s,
+                 :short => false
+              },
+              {
+                 :title => "First Noticed",
+                 :value => problem.first_notice_at,
+                 :short => false
+              },
+              {
+                 :title => "Last Noticed",
+                 :value => problem.last_notice_at,
+                 :short => false
+              },
+              {
+                 :title => "Times Occurred",
+                 :value => problem.notices_count,
+                 :short => false
+              }
+           ]
+        }
+      ]
+    }.to_json
     expect(HTTParty).to receive(:post).with(notification_service.service_url, :body => payload, :headers => {"Content-Type" => "application/json"}).and_return(true)
 
     notification_service.create_notification(problem)