Skip to content

[Messenger] Ability to use transactions while sending messages with AMQP #51719

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: 7.4
Choose a base branch
from
Open

[Messenger] Ability to use transactions while sending messages with AMQP #51719

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
6064a73
[Messenger] Add option option `transactional` to publish on AMQP queu…
michalsitek Sep 22, 2023
14de41d
Fixed changed version in changelog
michalsitek Sep 22, 2023
f290f46
Removed redundant sprintf
michalsitek Sep 22, 2023
e58dfd0
Removed redundant newline
michalsitek Sep 22, 2023
6c81d71
Made sure transaction is started before and committed after publishing
michalsitek Sep 22, 2023
ecb8749
Added missing return value for mock in tests
michalsitek Sep 22, 2023
41a9f2c
Removed double newlines
michalsitek Sep 22, 2023
dcb9fbf
Updated Symfony version in changelog
michalsitek Jun 3, 2025
a25a89e
Merge branch '7.4' into amqp-messenger-transactional-publish_51656
michalsitek Jun 3, 2025
f7aaff0
Removed unnecessary blank line in changelog
michalsitek Jun 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Added missing return value for mock in tests
  • Loading branch information
@michalsitek
michalsitek committed Sep 22, 2023
commit ecb8749e3b30a835232e750194ebd222e197240e
4 changes: 3 additions & 1 deletion src/Symfony/Component/Messenger/Bridge/Amqp/Tests/Transport/ConnectionTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -765,9 +765,11 @@ public function testItPublishMessagesWithoutTransactions()
$this->createMock(\AMQPConnection::class),
$amqpChannel = $this->createMock(\AMQPChannel::class),
$this->createMock(\AMQPQueue::class),
$this->createMock(\AMQPExchange::class)
$amqpExchange = $this->createMock(\AMQPExchange::class)
);

$amqpExchange->method('getChannel')->willReturn($amqpChannel);

$amqpChannel->expects($this->never())->method('startTransaction');
$amqpChannel->expects($this->never())->method('commitTransaction');

Expand Down
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy