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

Jump to bottom
Open
wants to merge 7 commits into
base: 7.3
Choose a base branch
from
Open

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

wants to merge 7 commits into from
+86 −0

Conversation

michalsitek
Copy link

@michalsitek michalsitek commented Sep 22, 2023 •
edited by OskarStark
Loading

Q A
Branch? 7.3
Bug fix? no
New feature? yes
Deprecations? no
Tickets Fix #51656
License MIT
Doc PR symfony/symfony-docs#18919

stloyd
stloyd reviewed Sep 22, 2023
View reviewed changes
michalsitek and others added 3 commits September 22, 2023 11:23
@michalsitek @stloyd
Fixed changed version in changelog
14de41d 
Co-authored-by: Joseph Bielawski <stloyd@users.noreply.github.com>
@michalsitek
Removed redundant sprintf
f290f46
@michalsitek
Removed redundant newline
e58dfd0
@michalsitek
Copy link
Author

Tests are failing due to:

There was 1 error:

1) Symfony\Component\Finder\Tests\FinderTest::testIgnoreVCSIgnoredWithOpenBasedir
PHPUnit\Framework\Exception: PHP Warning:  file_put_contents(): open_basedir restriction in effect. File(/tmp/phpunit_MB5isx) is not within the allowed path(s): (/home/runner/work/symfony/symfony:/tmp/symfony_finder/gitignore/search_root:/tmp/deprecLc2Yrz) in Standard input code on line 92
Warning: file_put_contents(): open_basedir restriction in effect. File(/tmp/phpunit_MB5isx) is not within the allowed path(s): (/home/runner/work/symfony/symfony:/tmp/symfony_finder/gitignore/search_root:/tmp/deprecLc2Yrz) in Standard input code on line 92
PHP Warning:  file_put_contents(/tmp/phpunit_MB5isx): Failed to open stream: Operation not permitted in Standard input code on line 92
Warning: file_put_contents(/tmp/phpunit_MB5isx): Failed to open stream: Operation not permitted in Standard input code on line 92

It seems to have nothing to do with my PR.

@nicolas-grekas nicolas-grekas added this to the 6.4 milestone Sep 25, 2023
@carsonbot carsonbot changed the title [Messenger] Ability to use transactions while sending messages with AMQP Ability to use transactions while sending messages with AMQP Sep 25, 2023
@carsonbot carsonbot changed the title Ability to use transactions while sending messages with AMQP [Messenger] Ability to use transactions while sending messages with AMQP Sep 25, 2023
@nicolas-grekas nicolas-grekas modified the milestones: 6.4, 7.1 Nov 15, 2023
@xabbuh xabbuh modified the milestones: 7.1, 7.2 May 15, 2024
@fabpot fabpot modified the milestones: 7.2, 7.3 Nov 20, 2024
@jwage jwage mentioned this pull request Apr 18, 2025
Closed
@jwage
Copy link
Contributor

jwage commented Apr 22, 2025

@michalsitek I added support for this in jwage/phpamqplib-messenger here jwage/phpamqplib-messenger#57

Could you take a look and give it a test?

OskarStark
OskarStark reviewed Apr 24, 2025
View reviewed changes
src/Symfony/Component/Messenger/Bridge/Amqp/CHANGELOG.md
@@ -1,6 +1,11 @@
CHANGELOG
=========

6.4
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
6.4
7.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stloyd stloyd stloyd left review comments

@OskarStark OskarStark OskarStark left review comments

Assignees
No one assigned
Labels
Feature Messenger
Projects
None yet
Milestone
7.3
Development

Successfully merging this pull request may close these issues.

[Messenger] Ability to use transactions while sending messages with AMQP transport
7 participants
@michalsitek @jwage @stloyd @OskarStark @fabpot @nicolas-grekas @xabbuh