From bc07e5f1c62320d117f8cab88cb610aeebdea64e Mon Sep 17 00:00:00 2001
From: Casper Thygesen <73483987+cthtrifork@users.noreply.github.com>
Date: Tue, 8 Nov 2022 19:48:03 +0100
Subject: [PATCH] Add default errorHandler() to vault-backend

Right now any uncaught error causes backstarte to crash
We should use `packages/backend-common/src/middleware/errorHandler.ts` middleware like the other backend plugins

Signed-off-by: Casper Thygesen <73483987+cthtrifork@users.noreply.github.com>
Signed-off-by: cthtrifork <cth@trifork.com>
---
 plugins/vault-backend/src/service/VaultBuilder.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/plugins/vault-backend/src/service/VaultBuilder.ts b/plugins/vault-backend/src/service/VaultBuilder.ts
index 67260f45ca..b82a5802bb 100644
--- a/plugins/vault-backend/src/service/VaultBuilder.ts
+++ b/plugins/vault-backend/src/service/VaultBuilder.ts
@@ -20,6 +20,7 @@ import { Logger } from 'winston';
 import express, { Router } from 'express';
 import { VaultClient } from './vaultApi';
 import { TaskRunner, PluginTaskScheduler } from '@backstage/backend-tasks';
+import { errorHandler } from '@backstage/backend-common';
 
 /**
  * Environment values needed by the VaultBuilder
@@ -145,6 +146,7 @@ export class VaultBuilder {
       res.json({ items: secrets });
     });
 
+    router.use(errorHandler());
     return router;
   }
 }