use de-JSON'd objects instead of raw strings

pjhyett · Jan 27, 2010 · 0c040e2 · 0c040e2
1 parent 9e25089
commit 0c040e2
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/github-services.rb b/github-services.rb
@@ -54,12 +54,13 @@ def service(name)
           yield data, payload
         end
       rescue => boom
-        hook_data = params[:data].dup
+        # redact sensitive info in hook_data hash
+        hook_data = data || params[:data]
         %w[password token].each { |key| hook_data[key] &&= '<redacted>' }
         report_exception boom,
           :hook_name    => name,
-          :hook_data    => hook_data,
-          :hook_payload => params[:payload]
+          :hook_data    => hook_data.inspect,
+          :hook_payload => (payload || params[:payload]).inspect
         raise
       end
     end