symfony / mercure

The Mercure Component allows to easily push updates to web browsers and other HTTP clients using the Mercure protocol.
https://symfony.com/doc/current/components/mercure.html
MIT License
413 stars 39 forks source link

Return result from Messenger UpdateHandler for HandleMessageMiddleware #87

Closed cvergne closed 2 years ago

cvergne commented 2 years ago

Hello,

I was looking for the result of the update with the HandledStamp, but it was always empty. Then I realized the result was not returned by the UpdateHandler, so the middleware could not set the result properly.

So the purpose of this PR is to let the UpdateHandler return the result from the Hub/Publisher to retrieve it from the HandledStamp.

The UpdateHandler test has also been updated to test if the publisher returns the expected result from HTTP response.

chalasr commented 2 years ago

Thank you @cvergne.