Backport #483 to be released as 1.7.1

(cherry picked from commit e2f3c47)

Author: danydev

Diff for: lib/Raven/Processor/SanitizeDataProcessor.php

@@ -79,7 +79,7 @@ public function sanitize(&$item, $key)
     public function sanitizeException(&$data)
     {
         foreach ($data['exception']['values'] as &$value) {
-            return $this->sanitizeStacktrace($value['stacktrace']);
+            $this->sanitizeStacktrace($value['stacktrace']);
         }
     }
 

Diff for: test/Raven/Tests/Processor/SanitizeDataProcessorTest.php

@@ -199,4 +199,53 @@ public static function overrideDataProvider()
             array($processorOptions, $client_options, $dsn)
         );
     }
+
+    public function testDoesFilterExceptionDataWithMultipleValues()
+    {
+        // Prerequisite: create an array with an 'exception' that contains 2 entry for 'values' key both containing at
+        // least 1 key that must be masked (i.e. 'password') in one of their 'vars' array in 'frames'.
+        $data = array(
+            'exception' => array(
+                'values' => array(
+                    array(
+                        'stacktrace' => array(
+                            'frames' => array(
+                                array(
+                                    'vars' => array(
+                                        'credentials' => array(
+                                            'password' => 'secretPassword'
+                                        ),
+                                    ),
+                                ),
+                            ),
+                        ),
+                    ),
+                    array(
+                        'stacktrace' => array(
+                            'frames' => array(
+                                array(
+                                    'vars' => array(
+                                        'credentials' => array(
+                                            'password' => 'anotherSecretPassword'
+                                        ),
+                                    ),
+                                ),
+                            ),
+                        ),
+                    ),
+                ),
+            ),
+        );
+
+        $client = new Dummy_Raven_Client();
+        $processor = new Raven_Processor_SanitizeDataProcessor($client);
+        // Action
+        $processor->process($data);
+
+        // Expectation: make sure we mask password in both the values array
+        $passwordValue0 = $data['exception']['values'][0]['stacktrace']['frames'][0]['vars']['credentials']['password'];
+        $this->assertEquals(Raven_Processor_SanitizeDataProcessor::STRING_MASK, $passwordValue0);
+        $passwordValue1 = $data['exception']['values'][1]['stacktrace']['frames'][0]['vars']['credentials']['password'];
+        $this->assertEquals(Raven_Processor_SanitizeDataProcessor::STRING_MASK, $passwordValue1);
+    }
 }