Merge pull request #107 from agentsib/fix-processor-options-definition

Fix processorOptions configuration
getsentry · Feb 26, 2018 · 1710460 · 1710460
2 parents 45a1e26 + d944ca4
commit 1710460
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Fixed
+ - Fix `processorOptions` in yaml configuration
 
 ## 2.0.1 - 2018-01-31
 ### Fixed

diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php
@@ -107,7 +107,9 @@ public function getConfigTreeBuilder()
                             ->prototype('scalar')->end()
                         ->end()
                         ->arrayNode('processorOptions')
-                            ->prototype('scalar')->end()
+                            ->arrayPrototype()
+                                ->prototype('scalar')->end()
+                            ->end()
                         ->end()
                     ->end()
                 ->end();

diff --git a/test/DependencyInjection/SentryExtensionTest.php b/test/DependencyInjection/SentryExtensionTest.php
@@ -343,7 +343,12 @@ public function test_that_it_sets_all_sentry_options()
             'install_shutdown_handler' => false,
             'processors' => ['processor1', 'processor2'],
             'processorOptions' => [
-                'processorOption1' => 'asasdf',
+                'processor1' => [
+                    'processorOption1' => 'asasdf',
+                ],
+                'processor2' => [
+                    'processorOption2' => 'asasdf',
+                ],
             ],
         ];