Merge pull request #2 from groupme/hash-recipient

hash the recipient's number in the log
groupme · May 21, 2014 · e0b260f · e0b260f
2 parents dba7d48 + fe4e22f
commit e0b260f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lib/em-kannel/log_message.rb b/lib/em-kannel/log_message.rb
@@ -15,7 +15,7 @@ def message
         parts = [
           "CODE=#{@response.status}",
           "FROM=#{@message.from}",
-          "TO=#{@message.to}",
+          "TO=#{Digest::MD5.hexdigest(@message.to.delete(" "))}",
           "BODY=#{@message.body[0..100]}",
           "RESPONSE=#{@response.body}",
           "TIME=#{@response.duration}"

diff --git a/spec/em-kannel/client_spec.rb b/spec/em-kannel/client_spec.rb
@@ -51,7 +51,8 @@
           to_return(:status => 202, :body => uuid)
 
         EM::Kannel.logger.should_receive(:info).with(
-          /CODE=202 FROM=\+12125551212 TO=\+17185551212 BODY=testing RESPONSE=#{uuid} TIME=[0-9]/
+          /CODE=202 FROM=\+12125551212 TO=78c929715ef3d957ad835697ced0e295 BODY=testing RESPONSE=#{uuid} TIME=[0-9]/
+
         )
 
         EM.run_block {
@@ -73,7 +74,7 @@
           to_return(:status => 404, :body => "API not available")
 
         EM::Kannel.logger.should_receive(:info).with(
-          /CODE=404 FROM=\+12125551212 TO=\+17185551212 BODY=testing RESPONSE=API not available TIME=[0-9]/
+          /CODE=404 FROM=\+12125551212 TO=78c929715ef3d957ad835697ced0e295 BODY=testing RESPONSE=API not available TIME=[0-9]/
         )
 
         EM.run_block {