Merge pull request #315 from beryllium/patch-1

[Sqs] Allow array-based DSN configuration

Author: makasim

pkg/sqs/SqsConnectionFactory.php

@@ -42,6 +42,12 @@ public function __construct($config = 'sqs:')
         } elseif (is_string($config)) {
             $config = $this->parseDsn($config);
         } elseif (is_array($config)) {
+            $dsn = array_key_exists('dsn', $config) ? $config['dsn'] : null;
+            unset($config['dsn']);
+
+            if ($dsn) {
+                $config = array_replace($config, $this->parseDsn($dsn));
+            }
         } else {
             throw new \LogicException('The config must be either an array of options, a DSN string or null');
         }

pkg/sqs/Tests/SqsConnectionFactoryConfigTest.php

@@ -104,6 +104,19 @@ public static function provideConfigs()
             ],
         ];
 
+        yield [
+            ['dsn' => 'sqs:?key=theKey&secret=theSecret&token=theToken&lazy=0'],
+            [
+                'key' => 'theKey',
+                'secret' => 'theSecret',
+                'token' => 'theToken',
+                'region' => null,
+                'retries' => 3,
+                'version' => '2012-11-05',
+                'lazy' => false,
+            ],
+        ];
+
         yield [
             ['key' => 'theKey', 'secret' => 'theSecret', 'token' => 'theToken', 'lazy' => false],
             [