Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Messenger] Allow to use custom http client for sqs messenger transport#61811

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.

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

Already on GitHub?Sign in to your account

Open
WaylandAce wants to merge2 commits intosymfony:7.4
base:7.4
Choose a base branch
Loading
fromWaylandAce:7.4

Conversation

WaylandAce
Copy link
Contributor

QA
Branch?7.4
Bug fix?no
New feature?no
Deprecations?no
LicenseMIT

In some scenarios (test cases) we need to pass specific HTTP client to messenger transport.

@carsonbotcarsonbot added this to the7.4 milestoneSep 22, 2025
@carsonbotcarsonbot changed the title[messenger] Allow to use custom http client for sqs messenger transport[Messenger] Allow to use custom http client for sqs messenger transportSep 22, 2025
@stof
Copy link
Member

The integration in FrameworkBundle should probably be updated to use it, so that it benefits of all the http_client configuration in FrameworkBundle.

@WaylandAce
Copy link
ContributorAuthor

@stof It could lead to BC problems, therefore I decided to keep it as is.

@nicolas-grekas
Copy link
Member

Which kind of problem?
To me not having thehttp_client service injected means we won't see requests in the profiler.
This PR is the opportunity to fix this inconsistency \o/

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

@nicolas-grekasnicolas-grekasnicolas-grekas approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
7.4
Development

Successfully merging this pull request may close these issues.

5 participants
@WaylandAce@stof@nicolas-grekas@carsonbot@pavelkirpitsov
[8]ページ先頭
©2009-2025 Movatter.jp