Merge pull request #1 from tombojer/master

Add health endpoint
niklasdoerfler · Jun 23, 2023 · 8c0084d · 8c0084d
2 parents 94823d3 + 1ef46f1
commit 8c0084d
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/app.js b/app.js
@@ -5,6 +5,21 @@ const dns = require('dns');
 const app = express();
 const port = 3000;
 
+
+app.get('/health', (req, res, next) => {
+    const target = process.env.TARGET_HOSTS_DNS_NAME;
+    if (target == undefined) {
+        return next(new Error("environment variable TARGET_HOSTS_DNS_NAME is not set"))
+    }
+
+    dns.lookup(process.env.TARGET_HOSTS_DNS_NAME, { all: true }, (err, addresses) => {
+        if (addresses == undefined) {
+            return next(new Error("Unable to resolve hostname '" + process.env.TARGET_HOSTS_DNS_NAME + "'."));
+        }
+        res.send(addresses)
+    })
+})
+
 app.all('*', (req, res, next) => {
     dns.lookup(process.env.TARGET_HOSTS_DNS_NAME, { all: true }, (err, addresses) => {
         if (addresses == undefined) {

diff --git a/helm/templates/deployment.yaml b/helm/templates/deployment.yaml
@@ -39,11 +39,11 @@ spec:
               protocol: TCP
           livenessProbe:
             httpGet:
-              path: /
+              path: /health
               port: http
           readinessProbe:
             httpGet:
-              path: /
+              path: /health
               port: http
 
           # environment configuration variables