Described DeduplicateStamp - #3317
Conversation
Preview of modified filesPreview of modified Markdown: |
Summary
|
There was a problem hiding this comment.
This is not entirely true. Our ibexa.messenger.bus supports only default middleware/stamps with basic functionality of messenger + our implemented middlewares/stamps. What it supports is explicitly configured by us, and below is the current list:
- all Symfony default middlewares described in https://symfony.com/doc/7.4/messenger.html#middleware. Additionally to this list, we support the technical middleware https://github.com/symfony/messenger/blob/7.4/Middleware/RejectRedeliveredMessageMiddleware.php, but I don't think we need to mention that in doc. Also, since Symfony 8.1 there's a new default
decode_failed_message, but we don't support it (https://symfony.com/doc/current/messenger.html#middleware). - 3 our stamps are already mentioned
shortly, I wouldn't mention that we support (all) stamps available in Symfony because it's not true. All supported Symfony stamps are the most basic ones, which deliver only basic messenger functionality.
There was a problem hiding this comment.
Thanks! I've also asked Claude to analyze this, and all of this combined (with me checking the Claude results ofc) results in:
Hope it's ok now
There was a problem hiding this comment.
I think it's correct now.
However, @Steveb-p could you double-check that I didn't miss anything here, please? :)
There was a problem hiding this comment.
We might consider working on it some more to allow adding of other Symfony middleware, especially since currently we need to perform manual adjustments.
code_samples/ change report
|
There was a problem hiding this comment.
We do add support for it in 6.x, right? I don't remember what we agreed on eventually :D
There was a problem hiding this comment.
In 5.x, we keep our implementation of stamp and middleware.
In 6.x, we replace it with SF implementation 😉
There was a problem hiding this comment.
Yes, I'll make this change when cherry-picking this to 6.0:
#3317 (comment)
And the deprecation is mentioned already as well
julitafalcondusza
left a comment
There was a problem hiding this comment.
Two small remarks.
There was a problem hiding this comment.
| 1. Inject the `ibexa.messenger.bus` service as an object implementing the `Symfony\Component\Messenger\MessageBusInterface` interface. |
There was a problem hiding this comment.
| The following example shows how you can attach the `DeduplicateStamp` to the message: |
Target: 5.0, 4.6, 6.0
Some minor changes for 4.6 will be needed (adjusting the version)
In 6.0 Deduplicate stamp is removed, so the whole section can be removed.
The deprecation is mentioned in #3279 already