From fc1b260bcbfe37aede11c866a4ea39e23b5ad099 Mon Sep 17 00:00:00 2001
From: Brandon Liu <bdon@users.noreply.github.com>
Date: Thu, 9 May 2024 10:44:56 +0800
Subject: [PATCH] detect relative URL pmtiles correctly [#152] (#155)

* detect relative URL pmtiles correctly [#152]

* put in a placeholder URL to let URL() parse relative URLs without http prefix.

* js 3.1.2; update changelog

* fix examples
---
 CHANGELOG.md                  | 9 +++++++++
 benchmark/index.html          | 2 +-
 examples/comparison.html      | 2 +-
 examples/fonts.html           | 2 +-
 examples/inset.html           | 2 +-
 examples/labels.html          | 2 +-
 examples/leaflet.html         | 2 +-
 examples/multi_source.html    | 2 +-
 examples/pmtiles.html         | 2 +-
 examples/pmtiles_headers.html | 2 +-
 examples/sandwich.html        | 2 +-
 examples/sprites.html         | 2 +-
 examples/static.html          | 2 +-
 package-lock.json             | 4 ++--
 package.json                  | 2 +-
 src/view.ts                   | 5 +----
 test/view.test.ts             | 5 +++++
 17 files changed, 30 insertions(+), 19 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index db985b9e..16c1d12e 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,12 @@
+# 3.1.2
+* Fix pmtiles URL detection on relative paths [#152]
+
+# 3.1.1
+* Detect pmtiles URLs by using URL parsing, which handles query parameters [#147]
+
+# 3.1.0
+* add `queryTileFeaturesDebug` back in for basic interactions (You should use MapLibre if you want interactivity)
+
 # 3.0.0
 
 * Unexport `PMTiles` because you shouldn't be depending on the one bundled in this library.
diff --git a/benchmark/index.html b/benchmark/index.html
index 3235e2cd..317da75c 100644
--- a/benchmark/index.html
+++ b/benchmark/index.html
@@ -6,7 +6,7 @@
         <meta name="viewport" content="width=device-width, initial-scale=1">
         <script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js"></script>
         <script src="pixelmatch.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             html {
diff --git a/examples/comparison.html b/examples/comparison.html
index 892fdb21..043ed1b2 100644
--- a/examples/comparison.html
+++ b/examples/comparison.html
@@ -9,7 +9,7 @@
         <link rel="stylesheet" href="https://unpkg.com/maplibre-gl@3.3.1/dist/maplibre-gl.css" crossorigin="anonymous">
         <script src="https://unpkg.com/maplibre-gl@3.3.1/dist/maplibre-gl.js" crossorigin="anonymous"></script>
         <script src="https://unpkg.com/@maplibre/maplibre-gl-leaflet@0.0.20/leaflet-maplibre-gl.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <style>
             #parent {
                 display:flex;
diff --git a/examples/fonts.html b/examples/fonts.html
index e658d8b5..bd242d03 100644
--- a/examples/fonts.html
+++ b/examples/fonts.html
@@ -5,7 +5,7 @@
         <link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css"/>
         <script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js"></script>
         <script src="https://unpkg.com/leaflet-hash@0.2.1/leaflet-hash.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
 
         <link href="https://fonts.googleapis.com/css2?family=Inter:wght@100..900&family=Work+Sans:wght@100..900&family=Petrona:wght@100..900&family=Raleway:wght@100..900" rel="stylesheet">
diff --git a/examples/inset.html b/examples/inset.html
index 222b1c05..317ed9a7 100644
--- a/examples/inset.html
+++ b/examples/inset.html
@@ -2,7 +2,7 @@
     <head>
         <meta charset="utf-8" />
         <meta name="viewport" content="width=device-width, initial-scale=1">
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             #map {
diff --git a/examples/labels.html b/examples/labels.html
index 31b10b70..a6cd8d5f 100644
--- a/examples/labels.html
+++ b/examples/labels.html
@@ -5,7 +5,7 @@
         <link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css"/>
         <script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js"></script>
         <script src="https://unpkg.com/leaflet-hash@0.2.1/leaflet-hash.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             body, #map {
diff --git a/examples/leaflet.html b/examples/leaflet.html
index 623a009d..7e952100 100644
--- a/examples/leaflet.html
+++ b/examples/leaflet.html
@@ -5,7 +5,7 @@
         <link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css"/>
         <script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js"></script>
         <script src="https://unpkg.com/leaflet-hash@0.2.1/leaflet-hash.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             body, #map {
diff --git a/examples/multi_source.html b/examples/multi_source.html
index ad5ce78f..7b38695b 100644
--- a/examples/multi_source.html
+++ b/examples/multi_source.html
@@ -5,7 +5,7 @@
         <link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css"/>
         <script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js"></script>
         <script src="https://unpkg.com/leaflet-hash@0.2.1/leaflet-hash.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             body, #map {
diff --git a/examples/pmtiles.html b/examples/pmtiles.html
index 3bb41fe0..264d6539 100644
--- a/examples/pmtiles.html
+++ b/examples/pmtiles.html
@@ -4,7 +4,7 @@
         <meta name="viewport" content="width=device-width, initial-scale=1">
         <link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css"/>
         <script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             body, #map {
diff --git a/examples/pmtiles_headers.html b/examples/pmtiles_headers.html
index f2135ce4..27bcc512 100644
--- a/examples/pmtiles_headers.html
+++ b/examples/pmtiles_headers.html
@@ -5,7 +5,7 @@
         <link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css"/>
         <script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js"></script>
         <script src="https://unpkg.com/pmtiles@3.0.3/dist/pmtiles.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             body, #map {
diff --git a/examples/sandwich.html b/examples/sandwich.html
index 0ff9e405..1fe3467f 100644
--- a/examples/sandwich.html
+++ b/examples/sandwich.html
@@ -5,7 +5,7 @@
         <link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css"/>
         <script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js"></script>
         <script src="https://unpkg.com/leaflet-hash@0.2.1/leaflet-hash.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             body, #map {
diff --git a/examples/sprites.html b/examples/sprites.html
index bb68096c..e0340d02 100644
--- a/examples/sprites.html
+++ b/examples/sprites.html
@@ -5,7 +5,7 @@
         <link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css"/>
         <script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js"></script>
         <script src="https://unpkg.com/leaflet-hash@0.2.1/leaflet-hash.js"></script>
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             body, #map {
diff --git a/examples/static.html b/examples/static.html
index 7d0b549e..b25b6641 100644
--- a/examples/static.html
+++ b/examples/static.html
@@ -2,7 +2,7 @@
     <head>
         <meta charset="utf-8" />
         <meta name="viewport" content="width=device-width, initial-scale=1">
-        <script src="https://unpkg.com/protomaps-leaflet@3.1.1/dist/protomaps-leaflet.min.js"></script>
+        <script src="https://unpkg.com/protomaps-leaflet@3.1.2/dist/protomaps-leaflet.min.js"></script>
         <!-- <script src="../dist/protomaps-leaflet.js"></script> -->
         <style>
             #map {
diff --git a/package-lock.json b/package-lock.json
index 2f3d8e08..40cb0c23 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "protomaps-leaflet",
-  "version": "3.1.1",
+  "version": "3.1.2",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "protomaps-leaflet",
-      "version": "3.1.1",
+      "version": "3.1.2",
       "license": "BSD-3-Clause",
       "dependencies": {
         "@mapbox/point-geometry": "^0.1.0",
diff --git a/package.json b/package.json
index ff9569d7..b7846b14 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "protomaps-leaflet",
-  "version": "3.1.1",
+  "version": "3.1.2",
   "files": [
     "dist",
     "src"
diff --git a/src/view.ts b/src/view.ts
index bbc47f49..a66162af 100644
--- a/src/view.ts
+++ b/src/view.ts
@@ -241,10 +241,7 @@ export const sourcesToViews = (options: SourceOptions) => {
     const maxDataZoom = o.maxDataZoom || 15;
     let source: TileSource;
     if (typeof o.url === "string") {
-      if (
-        o.url.startsWith("http") &&
-        new URL(o.url).pathname.endsWith(".pmtiles")
-      ) {
+      if (new URL(o.url, "http://example.com").pathname.endsWith(".pmtiles")) {
         source = new PmtilesSource(o.url, true);
       } else {
         source = new ZxySource(o.url, true);
diff --git a/test/view.test.ts b/test/view.test.ts
index 75987d26..bcbee2f1 100644
--- a/test/view.test.ts
+++ b/test/view.test.ts
@@ -107,3 +107,8 @@ test("sources to views with pmtiles parameters", async () => {
   const v = sourcesToViews({ url: "http://example.com/foo.pmtiles?k=v" });
   assert.ok(v.get("").tileCache.source instanceof PmtilesSource);
 });
+
+test("sources to views local param", async () => {
+  const v = sourcesToViews({ url: "foo.pmtiles" });
+  assert.ok(v.get("").tileCache.source instanceof PmtilesSource);
+});