Skip to content
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

MessageFailed events do not contain message body if full-text indexing is disabled #2626

Closed
SzymonPobiega opened this issue Sep 20, 2021 · 0 comments · Fixed by #2628
Closed

MessageFailed events do not contain message body if full-text indexing is disabled #2626

SzymonPobiega opened this issue Sep 20, 2021 · 0 comments · Fixed by #2628
Assignees
@SzymonPobiega
Labels
candidate-for-next-release Improvement Small Task Item that is put on the Small Tasks board
Milestone
4.21.0

Comments

@SzymonPobiega
Copy link
Member

The MessageFailed event publisher used to include the message body in the event payload if it was serialized into a non-binary form and is under 85 KB of size.

After the full-text indexing toggle was introduced the logic changed and the body is included only if the indexing is enabled.

It would be nice if the original logic was put in place.

NOTE: There is no plan to include the body in cases where it was not present before i.e. when serialized as binary or when over 85 KB.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SzymonPobiega SzymonPobiega

Labels
candidate-for-next-release Improvement Small Task Item that is put on the Small Tasks board
Projects
None yet
Milestone
4.21.0
Development

Successfully merging a pull request may close this issue.

Include body in MessageFailed event when full-text indexing is disabled Particular/ServiceControl
1 participant
@SzymonPobiega