OpenConext / Monitor-bundle

Openconext bundle for monitoring endpoints
Apache License 2.0
5 stars 2 forks source link

Publish /interal/info and /interal/health routes #8

Closed MKodde closed 2 years ago

MKodde commented 2 years ago

The /info and /health are to be replaced by the internal counterparts.

Using https://symfony.com/blog/new-in-symfony-5-4-route-aliasing#deprecating-routes would have been nice. But we'll just have to remember removing the old routes in the next major release.

https://www.pivotaltracker.com/story/show/181884474

codecov[bot] commented 2 years ago

Codecov Report

Merging #8 (8db90ed) into master (3f36539) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #8   +/-   ##
=========================================
  Coverage     42.17%   42.17%           
  Complexity       52       52           
=========================================
  Files            12       12           
  Lines           147      147           
=========================================
  Hits             62       62           
  Misses           85       85           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3f36539...8db90ed. Read the comment docs.

thijskh commented 2 years ago

Probably the README needs updating as well?