Catch the error from the fsa.readJson

linz · Oct 3, 2023 · 9c69baa · 9c69baa
1 parent 8155592
commit 9c69baa
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 16 deletions.
diff --git a/packages/lambda-tiler/src/util/__test__/config.loader.test.ts b/packages/lambda-tiler/src/util/__test__/config.loader.test.ts
@@ -82,8 +82,8 @@ o.spec('ConfigLoader', () => {
       .catch((e) => e);
 
     o(error instanceof LambdaHttpResponse).equals(true);
-    o((error as LambdaHttpResponse).status).equals(404);
-    o((error as LambdaHttpResponse).statusDescription).equals(`Config not found at ${location}`);
+    o((error as LambdaHttpResponse).status).equals(400);
+    o((error as LambdaHttpResponse).statusDescription).equals(`Invalid config location at ${location}`);
   });
 
   o('should get expected config file', async () => {
@@ -115,15 +115,15 @@ o.spec('ConfigLoader', () => {
   });
 
   const deletedLocation = 'memory://linz-basemaps/config-deleted.json';
-  o('should Error 404 if config file not exists', async () => {
+  o('should Error 400 if config file not exists', async () => {
     const error = await ConfigLoader.load(
       mockUrlRequest('/v1/tiles/🦄 🌈/NZTM2000Quad/tile.json', `?config=${deletedLocation}`, Api.header),
     )
       .then(() => null)
       .catch((e) => e);
 
     o(error instanceof LambdaHttpResponse).equals(true);
-    o((error as LambdaHttpResponse).status).equals(404);
-    o((error as LambdaHttpResponse).statusDescription).equals(`Config not found at ${deletedLocation}`);
+    o((error as LambdaHttpResponse).status).equals(400);
+    o((error as LambdaHttpResponse).statusDescription).equals(`Invalid config location at ${deletedLocation}`);
   });
 });
diff --git a/packages/lambda-tiler/src/util/config.cache.ts b/packages/lambda-tiler/src/util/config.cache.ts
@@ -5,9 +5,10 @@ import { SwappingLru } from './swapping.lru.js';
 class LruConfig {
   configProvider: Promise<ConfigProviderMemory | null>;
 
-  constructor(config: Promise<ConfigBundled>) {
+  constructor(config: Promise<ConfigBundled | null>) {
     this.configProvider = config
       .then((c) => {
+        if (c == null) return null;
         const configProvider = ConfigProviderMemory.fromJson(c);
         configProvider.createVirtualTileSets();
         return configProvider;
@@ -33,7 +34,9 @@ export class ConfigCache {
   get(location: string): Promise<ConfigProviderMemory | null> {
     const existing = this.cache.get(location)?.configProvider;
     if (existing != null) return existing;
-    const configJson = fsa.readJson<ConfigBundled>(location);
+    const configJson = fsa.readJson<ConfigBundled>(location).catch(() => {
+      return null;
+    });
     const config = new LruConfig(configJson);
     this.cache.set(location, config);
     return config.configProvider;

diff --git a/packages/lambda-tiler/src/util/config.loader.ts b/packages/lambda-tiler/src/util/config.loader.ts
@@ -46,14 +46,10 @@ export class ConfigLoader {
 
     req.set('config', configLocation);
     req.timer.start('config:load');
-    return CachedConfig.get(configLocation)
-      .then((f) => {
-        req.timer.end('config:load');
-        if (f == null) throw new LambdaHttpResponse(404, `Config not found at ${configLocation}`);
-        return f;
-      })
-      .catch((e) => {
-        throw new LambdaHttpResponse(e.status, e.statusDescription);
-      });
+    return CachedConfig.get(configLocation).then((f) => {
+      req.timer.end('config:load');
+      if (f == null) throw new LambdaHttpResponse(400, `Invalid config location at ${configLocation}`);
+      return f;
+    });
   }
 }