CORS support

[domino91]

No description provided.

[bpolaszek]

Hello there,

Sorry for taking so long to review this! 😬
If you're still eager to contribute, I required some minor changes.

Thanks a lot!
Ben

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (6fc70c7) to head (b56697e).
Report is 9 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##                main       #23   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity       138       152   +14     
===========================================
  Files             25        27    +2     
  Lines            396       439   +43     
===========================================
+ Hits             396       439   +43     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[bpolaszek]

Hi @domino91, thanks for getting back on this.
Can you please fix CI (composer linter:fix) and add an integration test?
You could try to issue a GET request to /.well-known/mercure without any topic: it should respond with a 400 Bad Request and you're supposed to see CORS headers in the response.

Some LOC remain uncovered: https://app.codecov.io/gh/bpolaszek/freddie/pull/23/blob/src/Hub/Middleware/CorsMiddleware.php

Thanks!

[bpolaszek]

Thank you @domino91 👍