Catch missing queue exeception on queueExists #63

sdufel · 2015-02-10T16:56:21Z

Fixes #62

scrutinizer-notifier · 2015-02-10T18:40:23Z

k-k · 2015-02-10T18:44:20Z

src/Provider/AwsProvider.php

-            return true;
-        }
+            if ($this->queueUrl = $result->get('QueueUrl')) {
+                return true;


Do you mind setting the queueUrl back into cache here? Its an attempt to keep us from having to hit the API to keep track of it, any time we do, let's reset the cache.

Sam Dufel added 2 commits February 10, 2015 08:24

Catch missing queue exeception on queueExists

0fad2ed

Add AWS check for topic existence

deb9137

k-k reviewed Feb 10, 2015
View reviewed changes

pmartelletti mentioned this pull request Mar 7, 2015

Allow creation of AWS topic on existing queues #66

Closed

k-k closed this Dec 18, 2015

k-k reopened this Dec 18, 2015

k-k merged commit deb9137 into uecode:master Dec 18, 2015

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Catch missing queue exeception on queueExists #63

Catch missing queue exeception on queueExists #63

sdufel commented Feb 10, 2015

scrutinizer-notifier commented Feb 10, 2015

k-k Feb 10, 2015

Catch missing queue exeception on queueExists #63

Catch missing queue exeception on queueExists #63

Conversation

sdufel commented Feb 10, 2015

scrutinizer-notifier commented Feb 10, 2015

k-k Feb 10, 2015

Choose a reason for hiding this comment