Skip to content

Conversation

mohamedachrefhacheni
Copy link

This PR introduces a health check endpoint to monitor service availability and database connectivity.
Impact:
Enables better monitoring
Helps DevOps with uptime checks
Follows REST API best practices

Copy link
Contributor

@YuriiMotov YuriiMotov left a comment

Choose a reason for hiding this comment

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

@mohamedachrefhacheni, thanks for your interest and efforts!

We already have health check endpoint:

@router.get("/health-check/")
async def health_check() -> bool:
return True

So, changes should be made there instead of adding one more.

Also, all changes should be covered by tests.

Would you like to continue working on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants