Commit d71b5bc
committed
minor #16720 [Messenger] Add doc for default routing for messages (clemherreman)
This PR was squashed before being merged into the 5.4 branch.
Discussion
----------
[Messenger] Add doc for default routing for messages
Hello,
I recently got bit on production after deploying a code change. We created a new `FooMessage` & `FooHandler`, but didn't update the `messenger.yaml` to include routing information.
This caused `FooMessage` to be directly handled by the HTTP process that dispatched the message, instead of the CLI, consumer process of Messenger. This lead to various issues: missing env var in HTTP context, lack of monitoring/stats as we only looks for messages metrics in the `consumer` container, not the `api` one :)
Looking at the code, I discovered you may define a default transport, but couldn't find it in the doc.
Commits
-------
955b393 [Messenger] Add doc for default routing for messages1 file changed
+7
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
261 | 261 | | |
262 | 262 | | |
263 | 263 | | |
264 | | - | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
| 269 | + | |
| 270 | + | |
265 | 271 | | |
266 | 272 | | |
267 | 273 | | |
| |||
0 commit comments