Merge pull request #109 from supabase/remove-health-check

feat: remove health check endpoint
supabase · May 29, 2023 · 7b15f96 · 7b15f96
2 parents d31f414 + 08186fa
commit 7b15f96
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/crates/base/src/server.rs b/crates/base/src/server.rs
@@ -46,11 +46,6 @@ impl Service<Request<Body>> for WorkerService {
         let fut = async move {
             let req_uri = req.uri().clone();
 
-            // if the request is for the health endpoint return a 200 OK response
-            if req_uri.path() == "/_internal/health" {
-                return Ok(Response::new(Body::empty()));
-            }
-
             let (res_tx, res_rx) = oneshot::channel::<Result<Response<Body>, hyper::Error>>();
             let msg = WorkerRequestMsg { req, res_tx };
 

diff --git a/examples/main/index.ts b/examples/main/index.ts
@@ -5,6 +5,15 @@ console.log('main function started');
 serve(async (req: Request) => {
 	const url = new URL(req.url);
 	const { pathname } = url;
+
+	// handle health checks
+	if (pathname === '/_internal/health') {
+		return new Response(
+			JSON.stringify({ 'message': 'ok' }),
+			{ status: 200, headers: { 'Content-Type': 'application/json' } },
+		);
+	}
+
 	const path_parts = pathname.split('/');
 	const service_name = path_parts[1];