Skip to content

Commit

Permalink
Release 1.229.0
Browse files Browse the repository at this point in the history
  • Loading branch information
dgafka authored and github-actions[bot] committed Aug 27, 2024
1 parent 3da6eb1 commit a7c3e73
Show file tree
Hide file tree
Showing 23 changed files with 48 additions and 49 deletions.
4 changes: 2 additions & 2 deletions packages/Amqp/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@
},
"require": {
"ext-amqp": "*",
"ecotone/enqueue": "~1.228.1",
"ecotone/enqueue": "~1.229.0",
"enqueue/amqp-ext": "^0.10.18",
"enqueue/dsn": "^0.10.4",
"enqueue/enqueue": "^0.10.0"
Expand All @@ -65,7 +65,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "amqp"
Expand Down
16 changes: 8 additions & 8 deletions packages/Amqp/src/AmqpAcknowledgeCallbackWraper.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@

use Ecotone\Enqueue\CachedConnectionFactory;
use Ecotone\Enqueue\EnqueueAcknowledgementCallback;
use Ecotone\Enqueue\ReconnectableConnectionFactory;
use Ecotone\Messaging\Endpoint\AcknowledgementCallback;
use Ecotone\Messaging\Handler\Logger\LoggingGateway;
use Exception;

final class AmqpAcknowledgeCallbackWraper implements AcknowledgementCallback
{
Expand All @@ -31,8 +31,8 @@ public function accept(): void
{
try {
$this->acknowledgementCallback->accept();
}catch (\Exception $exception) {
$this->loggingGateway->info("Failed to acknowledge message, disconnecting AMQP Connection in order to self-heal. Failure happen due to: " . $exception->getMessage(), exception: $exception);
} catch (Exception $exception) {
$this->loggingGateway->info('Failed to acknowledge message, disconnecting AMQP Connection in order to self-heal. Failure happen due to: ' . $exception->getMessage(), exception: $exception);

$this->connectionFactory->reconnect();
}
Expand All @@ -42,8 +42,8 @@ public function reject(): void
{
try {
$this->acknowledgementCallback->reject();
}catch (\Exception $exception) {
$this->loggingGateway->info("Failed to reject message, disconnecting AMQP Connection in order to self-heal. Failure happen due to: " . $exception->getMessage(), exception: $exception);
} catch (Exception $exception) {
$this->loggingGateway->info('Failed to reject message, disconnecting AMQP Connection in order to self-heal. Failure happen due to: ' . $exception->getMessage(), exception: $exception);

$this->connectionFactory->reconnect();
}
Expand All @@ -53,10 +53,10 @@ public function requeue(): void
{
try {
$this->acknowledgementCallback->requeue();
}catch (\Exception $exception) {
$this->loggingGateway->info("Failed to requeue message, disconnecting AMQP Connection in order to self-heal. Failure happen due to: " . $exception->getMessage(), exception: $exception);
} catch (Exception $exception) {
$this->loggingGateway->info('Failed to requeue message, disconnecting AMQP Connection in order to self-heal. Failure happen due to: ' . $exception->getMessage(), exception: $exception);

$this->connectionFactory->reconnect();
}
}
}
}
4 changes: 2 additions & 2 deletions packages/Amqp/src/AmqpInboundChannelAdapter.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

namespace Ecotone\Amqp;

use AMQPChannelException;
use AMQPConnectionException;
use Ecotone\Enqueue\CachedConnectionFactory;
use Ecotone\Enqueue\EnqueueHeader;
Expand All @@ -15,7 +16,6 @@
use Ecotone\Messaging\Endpoint\PollingConsumer\ConnectionException;
use Ecotone\Messaging\Handler\Logger\LoggingGateway;
use Ecotone\Messaging\Message;
use Ecotone\Messaging\MessageHeaders;
use Ecotone\Messaging\Support\MessageBuilder;
use Interop\Amqp\AmqpMessage;
use Interop\Queue\Consumer;
Expand Down Expand Up @@ -103,7 +103,7 @@ public function receiveWithTimeout(int $timeout = 0): ?Message
$subscriptionConsumer->consume($timeout ?: $this->receiveTimeoutInMilliseconds);

return $this->queueChannel->receive();
} catch (AMQPConnectionException|\AMQPChannelException $exception) {
} catch (AMQPConnectionException|AMQPChannelException $exception) {
$this->connectionFactory->reconnect();
throw new ConnectionException('Failed to connect to AMQP broker', 0, $exception);
}
Expand Down
4 changes: 3 additions & 1 deletion packages/Amqp/src/AmqpReconnectableConnectionFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
use Enqueue\AmqpExt\AmqpConnectionFactory;
use Enqueue\AmqpExt\AmqpConsumer;
use Enqueue\AmqpExt\AmqpContext;
use Exception;
use Interop\Queue\ConnectionFactory;
use Interop\Queue\Context;
use Interop\Queue\SubscriptionConsumer;
Expand Down Expand Up @@ -72,7 +73,8 @@ public function reconnect(): void
if ($this->subscriptionConsumer) {
try {
$this->subscriptionConsumer->unsubscribeAll();
}catch (\Exception) {}
} catch (Exception) {
}
}
/** @var AMQPConnection $connection */
$connection = $connectionProperty->getValue($this->connectionFactory);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,4 +14,4 @@ public function getTicket(): string
{
return $this->ticket;
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,8 @@

namespace Test\Ecotone\Amqp\Fixture\DistributedCommandBus\Receiver;

use Ecotone\Messaging\Attribute\Asynchronous;
use Ecotone\Messaging\Attribute\Parameter\Header;
use Ecotone\Modelling\Attribute\CommandHandler;
use Ecotone\Modelling\Attribute\Distributed;
use Ecotone\Modelling\Attribute\EventHandler;
use Ecotone\Modelling\Attribute\QueryHandler;
use Ecotone\Modelling\EventBus;
use Test\Ecotone\Amqp\Fixture\DistributedCommandBus\Receiver\Event\TicketCreated;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,4 +14,4 @@ public function channel()
{
return AmqpBackedMessageChannelBuilder::create('async');
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -37,4 +37,4 @@ public function getTicketsNotifications(): int
{
return count($this->ticketNotifications);
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ public function test_distributing_command_misses_heartbeat_and_reconnects(): voi
$executionPollingMetadata = ExecutionPollingMetadata::createWithDefaults()->withExecutionTimeLimitInMilliseconds(10000)->withStopOnError(false);
$userService = $this->bootstrapEcotone('user_service', ['Test\Ecotone\Amqp\Fixture\DistributedCommandBus\Publisher'], [new UserService()], amqpConfig: ['heartbeat' => 1]);
$ticketService = $this->bootstrapEcotone('ticket_service', ['Test\Ecotone\Amqp\Fixture\DistributedCommandBus\Receiver', 'Test\Ecotone\Amqp\Fixture\DistributedCommandBus\ReceiverEventHandler'], [new TicketServiceReceiver([0, 6, 0]), new TicketNotificationEventHandler([0, 6, 0]),
"logger" => new EchoLogger()
'logger' => new EchoLogger(),
], amqpConfig: ['heartbeat' => 1]);

$ticketService->run('ticket_service', $executionPollingMetadata);
Expand Down
3 changes: 1 addition & 2 deletions packages/Amqp/tests/Integration/DistributedEventBusTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
use Ecotone\Messaging\Channel\PollableChannel\GlobalPollableChannelConfiguration;
use Ecotone\Messaging\Config\ModulePackageList;
use Ecotone\Messaging\Config\ServiceConfiguration;
use Ecotone\Messaging\Endpoint\ExecutionPollingMetadata;
use Enqueue\AmqpExt\AmqpConnectionFactory;
use Test\Ecotone\Amqp\AmqpMessagingTest;
use Test\Ecotone\Amqp\Fixture\DistributedEventBus\AsynchronousEventHandler\TicketNotificationSubscriber;
Expand Down Expand Up @@ -67,7 +66,7 @@ public function test_distributing_event_and_publish_async_without_amqp_transacti
{
$userService = $this->bootstrapEcotone('user_service', ['Test\Ecotone\Amqp\Fixture\DistributedEventBus\Publisher'], [new UserService()], extensionObjects: [
GlobalPollableChannelConfiguration::createWithDefaults()
->withCollector(false)
->withCollector(false),
]);
$ticketService = $this->bootstrapEcotone(
'ticket_service',
Expand Down
6 changes: 3 additions & 3 deletions packages/Dbal/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
}
},
"require": {
"ecotone/enqueue": "~1.228.1",
"ecotone/enqueue": "~1.229.0",
"enqueue/dbal": "^0.10.17",
"doctrine/dbal": "^2.12.0|^3.0"
},
Expand All @@ -45,7 +45,7 @@
"doctrine/cache": "^1.0.0",
"doctrine/annotations": "^1.13",
"wikimedia/composer-merge-plugin": "^2.1",
"ecotone/jms-converter": "~1.228.1",
"ecotone/jms-converter": "~1.229.0",
"symfony/expression-language": "^6.0|^7.0"
},
"scripts": {
Expand All @@ -58,7 +58,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "dbal"
Expand Down
2 changes: 1 addition & 1 deletion packages/Ecotone/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "ecotone"
Expand Down
4 changes: 2 additions & 2 deletions packages/Enqueue/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
}
},
"require": {
"ecotone/ecotone": "~1.228.1",
"ecotone/ecotone": "~1.229.0",
"queue-interop/queue-interop": "^0.8",
"enqueue/dsn": "^0.10.4"
},
Expand All @@ -56,7 +56,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "enqueue"
Expand Down
3 changes: 2 additions & 1 deletion packages/Enqueue/src/EnqueueInboundChannelAdapter.php
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,8 @@ public function receiveWithTimeout(int $timeoutInMilliseconds = 0): ?Message
if ($this->isConnectionException($exception) || ($exception->getPrevious() && $this->isConnectionException($exception->getPrevious()))) {
try {
$this->connectionFactory->reconnect();
}catch (\Exception) {}
} catch (Exception) {
}

throw new ConnectionException('There was a problem while polling message channel', 0, $exception);
}
Expand Down
4 changes: 2 additions & 2 deletions packages/JmsConverter/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
}
},
"require": {
"ecotone/ecotone": "~1.228.1",
"ecotone/ecotone": "~1.229.0",
"jms/serializer": "^3.17",
"symfony/cache": "^5.4|^6.1|^7.0"
},
Expand All @@ -60,7 +60,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "jms-converter"
Expand Down
6 changes: 3 additions & 3 deletions packages/Laravel/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
}
},
"require": {
"ecotone/ecotone": "~1.228.1",
"ecotone/ecotone": "~1.229.0",
"laravel/framework": "^9.5.2|^10.0|^11.0"
},
"require-dev": {
Expand All @@ -50,7 +50,7 @@
"symfony/expression-language": "^6.0|^7.0",
"nesbot/carbon": "^2.71",
"moneyphp/money": "^4.1.0",
"ecotone/dbal": "~1.228.1"
"ecotone/dbal": "~1.229.0"
},
"extra": {
"laravel": {
Expand All @@ -59,7 +59,7 @@
]
},
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "laravel"
Expand Down
6 changes: 3 additions & 3 deletions packages/LiteApplication/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,8 @@
}
},
"require": {
"ecotone/ecotone": "~1.228.1",
"ecotone/jms-converter": "~1.228.1",
"ecotone/ecotone": "~1.229.0",
"ecotone/jms-converter": "~1.229.0",
"php-di/php-di": "^7.0.1"
},
"require-dev": {
Expand All @@ -50,7 +50,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "lite-application"
Expand Down
4 changes: 2 additions & 2 deletions packages/OpenTelemetry/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
}
},
"require": {
"ecotone/ecotone": "~1.228.1",
"ecotone/ecotone": "~1.229.0",
"open-telemetry/sdk": "^1.0.0"
},
"require-dev": {
Expand Down Expand Up @@ -64,7 +64,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "open-telemetry"
Expand Down
4 changes: 2 additions & 2 deletions packages/PdoEventSourcing/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
}
},
"require": {
"ecotone/dbal": "~1.228.1",
"ecotone/dbal": "~1.229.0",
"prooph/pdo-event-store": "^1.15.1"
},
"require-dev": {
Expand All @@ -51,7 +51,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "pdo-event-sourcing"
Expand Down
4 changes: 2 additions & 2 deletions packages/Redis/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
}
},
"require": {
"ecotone/enqueue": "~1.228.1",
"ecotone/enqueue": "~1.229.0",
"enqueue/redis": "^0.10.9"
},
"require-dev": {
Expand All @@ -60,7 +60,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "redis"
Expand Down
4 changes: 2 additions & 2 deletions packages/Sqs/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
}
},
"require": {
"ecotone/enqueue": "~1.228.1",
"ecotone/enqueue": "~1.229.0",
"enqueue/sqs": "^0.10.15",
"aws/aws-sdk-php": "<=3.269.5"
},
Expand All @@ -58,7 +58,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "sqs"
Expand Down
6 changes: 3 additions & 3 deletions packages/Symfony/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
],
"description": "Extends Ecotone with Symfony integration",
"require": {
"ecotone/ecotone": "~1.228.1",
"ecotone/ecotone": "~1.229.0",
"symfony/console": "^5.4|^6.0|^7.0",
"symfony/dependency-injection": ">=v5.4.19 <6.0.0 || >=v6.0.19 <6.1.0 || >=v6.1.11 <6.2.0 || >=v6.2.5 <7.0.0 || >=v7.0.0 <8.0.0",
"symfony/framework-bundle": "^5.4|^6.0|^7.0"
Expand All @@ -45,7 +45,7 @@
"symfony/expression-language": "^6.0|^7.0",
"symfony/messenger": "^5.4|^6.0|^7.0",
"wikimedia/composer-merge-plugin": "^2.1",
"ecotone/dbal": "~1.228.1"
"ecotone/dbal": "~1.229.0"
},
"conflict": {
"symfony/proxy-manager-bridge": "<5.4.0"
Expand Down Expand Up @@ -89,7 +89,7 @@
},
"extra": {
"branch-alias": {
"dev-main": "1.228.1-dev"
"dev-main": "1.229.0-dev"
},
"ecotone": {
"repository": "symfony"
Expand Down
Loading

0 comments on commit a7c3e73

Please sign in to comment.