chore(value-service): handle configuration error and write

Value service is biased to writing a value even if configuration cannot be retrieved. This means that definition schema is a best effort.

apps/value-service/src/values/router/value.spec.ts

@@ -785,6 +785,41 @@ describe('event router', () => {
       expect(res.send).toHaveBeenCalledWith(expect.arrayContaining([]));
       expect(next).not.toHaveBeenCalled();
     });
+
+    it('can write value on configuration error', async () => {
+      const req = {
+        tenantId,
+        user: {
+          tenantId,
+          id: 'test-reader',
+          roles: [ServiceUserRoles.Reader],
+        },
+        params: { namespace: 'test-service', name: 'test-value' },
+        query: {},
+        body: {},
+        getConfiguration: jest.fn(),
+      };
+      const res = {
+        send: jest.fn(),
+      };
+      const next = jest.fn();
+
+      req.getConfiguration.mockRejectedValueOnce(new Error('oh noes!'));
+
+      const value = {};
+      repositoryMock.writeValue.mockResolvedValueOnce({
+        tenantId,
+        timestamp: new Date(),
+        context: {},
+        correlationId: null,
+        value,
+      });
+
+      const handler = writeValue(loggerMock, eventServiceMock, repositoryMock);
+      await handler(req as unknown as Request, res as unknown as Response, next);
+      expect(eventServiceMock.send).not.toHaveBeenCalled();
+      expect(res.send).toHaveBeenCalledWith(expect.objectContaining({ value }));
+    });
   });
 
   describe('readMetrics', () => {

apps/value-service/src/values/router/value.ts

@@ -263,7 +263,21 @@ export function writeValue(logger: Logger, eventService: EventService, repositor
         user: `${user.name} (ID: ${user.id})`,
       });
 
-      const [namespaces] = await req.getConfiguration<Record<string, NamespaceEntity>>(tenantId);
+      let namespaces: Record<string, NamespaceEntity>;
+      try {
+        [namespaces] = await req.getConfiguration<Record<string, NamespaceEntity>>(tenantId);
+      } catch (err) {
+        // Catch and log error on configuration retrieval.
+        // The value service is biased to writing the value even if the definition retrieval fails.
+        logger.warn(
+          `Error encountered retrieving value configuration; value will be written without definition: ${err}`,
+          {
+            context: 'value-router',
+            tenantId: tenantId?.toString(),
+            user: `${user.name} (ID: ${user.id})`,
+          }
+        );
+      }
 
       const results = [];
       const valuesToWrite = Array.isArray(req.body) ? req.body : [req.body];