Fix for the client-side caching breaking GeoJSON sources (#82)

* RD-254 Fix for the client-side caching breaking GeoJSON (and video) sources * Add changelog entry
maptiler · Apr 24, 2024 · db79e06 · db79e06
1 parent 14baf8e
commit db79e06
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 ## DEVEL
 ### New Features
 ### Bug Fixes
+- Fix for the client-side caching breaking GeoJSON sources [#82](https://github.com/maptiler/maptiler-sdk-js/pull/82)
 ### Others
 
 ## NEXT

diff --git a/src/caching.ts b/src/caching.ts
@@ -23,7 +23,7 @@ export function localCacheTransformRequest(
     config.session &&
     reqUrl.host === defaults.maptilerApiHost
   ) {
-    if (resourceType == "Source") {
+    if (resourceType == "Source" && reqUrl.href.includes("tiles.json")) {
       return reqUrl.href.replace(
         "https://",
         `${LOCAL_CACHE_PROTOCOL_SOURCE}://`,
@@ -74,9 +74,11 @@ export function registerLocalCacheProtocol() {
       const response = await fetch(params.url, requestInit);
       const json = await response.json();
 
-      // move `Last-Modified` to query so it propagates to tile URLs
-      json.tiles[0] +=
-        "&last-modified=" + response.headers.get("Last-Modified");
+      if (json.tiles && json.tiles.length > 0) {
+        // move `Last-Modified` to query so it propagates to tile URLs
+        json.tiles[0] +=
+          "&last-modified=" + response.headers.get("Last-Modified");
+      }
 
       return {
         data: json,