Skip to content

[Messenger] unwrap ReceivedMessage in LoggingMiddleware to improve log detail #27066

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 26, 2018
Merged

[Messenger] unwrap ReceivedMessage in LoggingMiddleware to improve log detail #27066

merged 1 commit into from
Apr 26, 2018

Conversation

kbond
Copy link
Member

@kbond kbond commented Apr 26, 2018

Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

The change gives better visibility in your logs when consuming messages.

@nicolas-grekas nicolas-grekas added this to the 4.1 milestone Apr 26, 2018
@Tobion
Copy link
Contributor

Tobion commented Apr 26, 2018

Thank you @kbond.

@Tobion Tobion merged commit 78bb025 into symfony:master Apr 26, 2018
Tobion added a commit that referenced this pull request Apr 26, 2018
…to improve log detail (kbond)

This PR was merged into the 4.1-dev branch.

Discussion
----------

[Messenger] unwrap ReceivedMessage in LoggingMiddleware to improve log detail

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

The change gives better visibility in your logs when consuming messages.

Commits
-------

78bb025 [Messenger] unwrap ReceivedMessage in LoggingMiddleware to improve log detail
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants