Skip to content

Commit 715a430

Browse files
committed
32866: Updated php-amqplib/php-amqplib to the latest version 3.0.0
1 parent 3e1e772 commit 715a430

File tree

3 files changed

+39
-28
lines changed

3 files changed

+39
-28
lines changed

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@
6767
"monolog/monolog": "^1.17",
6868
"paragonie/sodium_compat": "^1.6",
6969
"pelago/emogrifier": "^5.0.0",
70-
"php-amqplib/php-amqplib": "~2.10.0",
70+
"php-amqplib/php-amqplib": "~3.0.0",
7171
"phpseclib/mcrypt_compat": "1.0.8",
7272
"phpseclib/phpseclib": "2.0.*",
7373
"ramsey/uuid": "~4.1.0",

composer.lock

Lines changed: 24 additions & 16 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/internal/Magento/Framework/Amqp/Queue.php

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,13 @@
55
*/
66
namespace Magento\Framework\Amqp;
77

8+
use Closure;
9+
use Exception;
810
use Magento\Framework\MessageQueue\ConnectionLostException;
911
use Magento\Framework\MessageQueue\EnvelopeInterface;
1012
use Magento\Framework\MessageQueue\QueueInterface;
11-
use PhpAmqpLib\Exception\AMQPProtocolConnectionException;
13+
use Magento\Framework\Phrase;
14+
use PhpAmqpLib\Exception\AMQPTimeoutException;
1215
use PhpAmqpLib\Message\AMQPMessage;
1316
use Magento\Framework\MessageQueue\EnvelopeFactory;
1417
use Psr\Log\LoggerInterface;
@@ -73,11 +76,11 @@ public function dequeue()
7376
/** @var AMQPMessage $message */
7477
try {
7578
$message = $channel->basic_get($this->queueName);
76-
} catch (AMQPProtocolConnectionException $e) {
79+
} catch (AMQPTimeoutException $exception) {
7780
throw new ConnectionLostException(
78-
$e->getMessage(),
79-
$e->getCode(),
80-
$e
81+
$exception->getMessage(),
82+
$exception->getCode(),
83+
$exception
8184
);
8285
}
8386

@@ -107,11 +110,11 @@ public function acknowledge(EnvelopeInterface $envelope)
107110
// @codingStandardsIgnoreStart
108111
try {
109112
$channel->basic_ack($properties['delivery_tag']);
110-
} catch (AMQPProtocolConnectionException $e) {
113+
} catch (Exception $exception) {
111114
throw new ConnectionLostException(
112-
$e->getMessage(),
113-
$e->getCode(),
114-
$e
115+
$exception->getMessage(),
116+
$exception->getCode(),
117+
$exception
115118
);
116119
}
117120
// @codingStandardsIgnoreEnd
@@ -135,7 +138,7 @@ public function subscribe($callback)
135138
// @codingStandardsIgnoreEnd
136139
$envelope = $this->envelopeFactory->create(['body' => $message->body, 'properties' => $properties]);
137140

138-
if ($callback instanceof \Closure) {
141+
if ($callback instanceof Closure) {
139142
$callback($envelope);
140143
} else {
141144
call_user_func($callback, $envelope);
@@ -165,7 +168,7 @@ public function reject(EnvelopeInterface $envelope, $requeue = true, $rejectionM
165168
// @codingStandardsIgnoreEnd
166169
if ($rejectionMessage !== null) {
167170
$this->logger->critical(
168-
new \Magento\Framework\Phrase('Message has been rejected: %message', ['message' => $rejectionMessage])
171+
new Phrase('Message has been rejected: %message', ['message' => $rejectionMessage])
169172
);
170173
}
171174
}

0 commit comments

Comments
 (0)