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

[FrameworkBundle][Messenger] Added RouterContextMiddleware#39688

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

Merged
nicolas-grekas merged 1 commit intosymfony:5.xfromjderusse:fwb-router-middleware
Feb 11, 2021

Conversation

@jderusse
Copy link
Member

QA
Branch?5.x
Bug fix?no
New feature?yes
Deprecations?no
Tickets-
LicenseMIT
Doc PRTODO

When handling a message in async, we, sometimes need the Router Context to generate absolute URL. ie:

  • sending an email when the message contains only the template
  • generating a PDF

People can use the configurationrouter.default_uri to workaround and fix the issue, but this does not work when the web application servers several domains.

This PR provide a new middleware that store the current router context in a stamp, and restore the context when processing the message.

@jderussejderusse added this to the5.x milestoneJan 2, 2021
@carsonbotcarsonbot changed the title[FrameworkBundle] Added RouterContextMiddleware[FrameworkBundle][Messenger] Added RouterContextMiddlewareJan 2, 2021
@jderussejderusseforce-pushed thefwb-router-middleware branch 4 times, most recently from0a55b33 tob3beab6CompareJanuary 3, 2021 16:34
@nicolas-grekas
Copy link
Member

Error: Call to undefined method Symfony\Component\Routing\RequestContext::fromUri()

on deps=low

@jderussejderusseforce-pushed thefwb-router-middleware branch 2 times, most recently fromc5ef324 to821fa53CompareFebruary 6, 2021 18:00
@nicolas-grekas
Copy link
Member

Thank you@jderusse.

@jderussejderusse deleted the fwb-router-middleware branchFebruary 11, 2021 10:29
@fabpotfabpot mentioned this pull requestApr 18, 2021
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

@derrabusderrabusderrabus approved these changes

@srozesrozeAwaiting requested review from sroze

Assignees

No one assigned

Projects

None yet

Milestone

5.4

Development

Successfully merging this pull request may close these issues.

4 participants

@jderusse@nicolas-grekas@derrabus@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp