Fix geojson return type

andrew-bierman · Dec 18, 2024 · dbbf6c8 · dbbf6c8
1 parent d173b04
commit dbbf6c8
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 12 deletions.
diff --git a/server/src/routes/geojsonRoutes.ts b/server/src/routes/geojsonRoutes.ts
@@ -18,17 +18,24 @@ router.get(
         params.resourceId,
       );
 
+      if (!object) {
+        return ctx.json(
+          {
+            error: `Resource ${params.resource}/${params.resourceId} not found`,
+          },
+          404,
+        );
+      }
+
       const headers = new Headers();
       object.writeHttpMetadata(headers);
-      headers.set('etag', object.httpEtag);
-      headers.set('Content-Type', 'application/geo+json');
+      ctx.header('etag', object.httpEtag);
+      ctx.header('Content-Type', 'application/geo+json');
       const json = await object.json();
 
-      return new Response(json, {
-        headers,
-      });
+      return ctx.json(json);
     } catch (error) {
-      return ctx.json({ error: error.message }, 400);
+      return ctx.json({ error: error.message }, 500);
     }
   }),
 );

diff --git a/server/src/services/geojsonStorage/index.ts b/server/src/services/geojsonStorage/index.ts
@@ -23,16 +23,14 @@ export class GeojsonStorageService {
     resource: ResourceType,
     geojson: string,
     resourceId: string,
-  ): Promise<R2Object> {
-    const object = await this._bucket.put(`${resource}/${resourceId}`, geojson);
-    return object;
+  ): Promise<R2Object | null> {
+    return this._bucket.put(`${resource}/${resourceId}`, geojson);
   }
 
   public static async retrieve(
     resource: ResourceType,
     resourceId: string,
-  ): Promise<R2ObjectBody> {
-    const object = await this._bucket.get(`${resource}/${resourceId}`);
-    return object;
+  ): Promise<R2ObjectBody | null> {
+    return this._bucket.get(`${resource}/${resourceId}`);
   }
 }
diff --git a/server/src/services/trip/editTripService.ts b/server/src/services/trip/editTripService.ts
@@ -37,6 +37,7 @@ export const editTripService = async (
     await scoreTripService(selectedTrip.id);
 
     const serializedGeoJSON = tripData.geoJSON;
+
     if (!serializedGeoJSON) {
       return updatedTrip;
     }