Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

monitoring: improve returned information #503

Merged
merged 1 commit into from
Feb 26, 2021
Merged

monitoring: improve returned information #503

merged 1 commit into from
Feb 26, 2021

Conversation

sebdeleze
Copy link
Contributor

  • Improves monitoring API endpoints.
  • Moves the authentication check to before_request hook.
  • Adds an endpoint to get the ES health.

Co-Authored-by: Sébastien Délèze sebastien.deleze@rero.ch

@sebdeleze sebdeleze marked this pull request as ready for review February 23, 2021 07:50
@sebdeleze sebdeleze requested a review from rerowep February 23, 2021 07:50
Copy link
Contributor

@rerowep rerowep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice to have some informations if there are errors like not authorized ...

* Improves monitoring API endpoints.
* Moves the authentication check to `before_request` hook.
* Adds an endpoint to get the ES health.
* Closes #502.

Co-Authored-by: Sébastien Délèze <sebastien.deleze@rero.ch>
@sebdeleze sebdeleze merged commit e0ac685 into rero:staging Feb 26, 2021
@sebdeleze sebdeleze deleted the sed-improve-monitoring branch February 26, 2021 10:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants