starred/hoppscotch

Fork 0

mirror of https://github.com/hoppscotch/hoppscotch.git synced 2026-04-25 16:55:59 +03:00

[GH-ISSUE #3830] [feature]: add health check checking the DB connection #1343

New issue

Closed

opened 2026-03-16 19:55:31 +03:00 by kerem · 4 comments

kerem commented

2026-03-16 19:55:31 +03:00

Owner

Originally created by @RaHehl on GitHub (Feb 15, 2024).
Original GitHub issue: https://github.com/hoppscotch/hoppscotch/issues/3830

Is there an existing issue for this?

I have searched the existing issues

Summary

I need a health check that also checks whether the database is accessible.
The current "ping" does not check whether the DB is reachable packages/hoppscotch-backend/src/app.controller.ts

A simple SELECT 1 would be enough

I currently have the problem that Hoppscotch is no longer accessible when the DB credentials are updated automatically, and the health check does not activate the new ones by restarting the pod because the health check is still successful.

Why should this be worked on?

see above

Originally created by @RaHehl on GitHub (Feb 15, 2024). Original GitHub issue: https://github.com/hoppscotch/hoppscotch/issues/3830 ### Is there an existing issue for this? - [X] I have searched the existing issues ### Summary I need a health check that also checks whether the database is accessible. The current "ping" does not check whether the DB is reachable packages/hoppscotch-backend/src/app.controller.ts A simple SELECT 1 would be enough I currently have the problem that Hoppscotch is no longer accessible when the DB credentials are updated automatically, and the health check does not activate the new ones by restarting the pod because the health check is still successful. ### Why should this be worked on? see above

kerem

2026-03-16 19:55:31 +03:00

closed this issue
added the
feature
label

kerem commented

2026-03-16 19:55:47 +03:00

Author

Owner

@Shub3am commented on GitHub (Feb 16, 2024):

Can i work on this?

@Shub3am commented on GitHub (Feb 16, 2024): Can i work on this?

kerem commented

2026-03-16 19:55:52 +03:00

Author

Owner

@balub commented on GitHub (Apr 29, 2024):

@RaHehl Could you give us more clarity on the expectations of this feature.

Please feel free to correct me if I got this wrong, but do you want the current /ping route also run a simple "SELECT 1` query against the DB to check if the backend instance can communicate with the DB, even after the DB creds change?

@balub commented on GitHub (Apr 29, 2024): @RaHehl Could you give us more clarity on the expectations of this feature. Please feel free to correct me if I got this wrong, but do you want the current `/ping` route also run a simple "SELECT 1` query against the DB to check if the backend instance can communicate with the DB, even after the DB creds change?

kerem commented

2026-03-16 19:55:57 +03:00

Author

Owner

@RaHehl commented on GitHub (Apr 29, 2024):

@balub
this does not necessarily have to be /ping, e.g. /health would be OK too.
The health check currently does not check whether the backend can establish a connection to the database.

Perhaps a clean solution would be to integrate Terminus, as described in the NestJS example:

https://docs.nestjs.com/recipes/terminus#healthchecks-terminus

https://github.com/nestjs/terminus/blob/master/sample/012-prisma-app/src/health/health.controller.ts

@RaHehl commented on GitHub (Apr 29, 2024): @balub this does not necessarily have to be /ping, e.g. /health would be OK too. The health check currently does not check whether the backend can establish a connection to the database. Perhaps a clean solution would be to integrate Terminus, as described in the NestJS example: https://docs.nestjs.com/recipes/terminus#healthchecks-terminus https://github.com/nestjs/terminus/blob/master/sample/012-prisma-app/src/health/health.controller.ts

kerem commented

2026-03-16 19:56:02 +03:00

Author

Owner

@balub commented on GitHub (Apr 29, 2024):

@RaHehl understood, I just wanted to make sure we were on the same page when talking about we wanted a way to check if the backend can talk to the database. We will look into Terminus looks promising.

@balub commented on GitHub (Apr 29, 2024): @RaHehl understood, I just wanted to make sure we were on the same page when talking about we wanted a way to check if the backend can talk to the database. We will look into Terminus looks promising.

kerem referenced this issue

2026-03-17 00:52:46 +03:00

[PR #1343] [MERGED] Request method labels & custom method width fix #3246

kerem referenced this issue

2026-03-17 00:53:03 +03:00

[PR #1350] [MERGED] Bump socket.io-client from 3.0.1 to 3.0.2 #3254