Merge pull request #17 from travel-spirit/fix/no-credentials

fix: when using roles only (no key and secret), remove also succeeds
palpalani · Jun 30, 2023 · 74c8de5 · 74c8de5
2 parents c8ae833 + f89548e
commit 74c8de5
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/src/SqsQueueReaderServiceProvider.php b/src/SqsQueueReaderServiceProvider.php
@@ -62,16 +62,23 @@ private function removeMessages(array $data, $queue): void
         $batchIds = array_column($data['data'], 'batchIds');
         $batchIds = array_chunk($batchIds, 10);
 
-        $client = new SqsClient([
+        $config = Config::get('queue.connections.sqs-json');
+
+        $sqsClientConfig = [
             //'profile' => 'default',
             'region' => Config::get('queue.connections.sqs-json.region'),
             'version' => '2012-11-05',
-            'credentials' => Arr::only(Config::get('queue.connections.sqs-json'), ['key', 'secret']),
             'http' => [
                 'timeout' => 30,
                 'connect_timeout' => 30,
             ],
-        ]);
+        ];
+
+        if ($config['key'] && $config['secret']) {
+            $sqsClientConfig['credentials'] = Arr::only($config, ['key', 'secret']);
+        }
+
+        $client = new SqsClient($sqsClientConfig);
 
         foreach ($batchIds as $batch) {
             //Deletes up to ten messages from the specified queue.