[simple-client] sqs: DSN not working

[jhogervorst]

I’ve installed enqueue/sqs, enqueue/fs, and enqueue/simple-client via Composer.

When running:

$dsn = 'sqs:?key=aKey&secret=aSecret&region=eu-west-1';
$client = new \Enqueue\SimpleClient\SimpleClient($dsn);

I get an exception:

Symfony\Component\Config\Definition\Exception\InvalidConfigurationException
Unrecognized option "sqs:?key=aKey&secret=aSecret&region=eu_west_1" under "enqueue.transport"

I’ve already checked whether the SQS broker exists, which is the case (var_dump(class_exists(\Enqueue\Sqs\SqsConnectionFactory::class)) yields bool(true)).

I did notice that when I change the DSN to sqs://... (so adding two slashes), I do not get the Symphony exception. However, then I get a "Failed to parse DSN" exception from SqsConnectionFactory, as it doesn’t expect those extra slashes

[jhogervorst]

Also, I already found the issue 😄

In enqueue/simple-client/SimpleClient.php:380, there is a check whether :// occurs in the DSN, which is not the case for the SQS DSN. If I manually specify the config array that would be used if :// did occur, it does work:

$dsn = 'sqs:?key=aKey&secret=aSecret&region=eu-west-1';
$config = ['transport' => ['default' => $dsn]];
$client = new \Enqueue\SimpleClient\SimpleClient($config);