pmtiles-swarm 0.45.2 → 0.46.0

package/CHANGELOG.md

@@ -7,6 +7,46 @@
 ### 🐞 Bug fixes
 - _...Add new stuff here..._
 
+## 0.46.0
+### ✨ Features and improvements
+- **Each category endpoint on the public page is now offered the way it is used.** A `.torrent` is a
+  file, so it downloads. A preview is a page, so it opens. The rest are addresses that belong
+  somewhere else — a style, a torrent client, a feed reader — and following one here achieves
+  nothing, so TileJSON, magnet, RSS and the style URL copy instead.
+
+  The style URL is no longer printed. It is a TileJSON URL carrying a `.torrent` URL and a
+  percent-encoded magnet in its fragment, several hundred characters of it, and it was most of the
+  card — while being the one thing on the page nobody reads and everybody pastes. Every copy button
+  also carries its URL as a tooltip, since a browser on plain HTTP has no clipboard and a string
+  somebody can select beats a button that does nothing.
+
+### 🐞 Bug fixes
+
+## 0.45.4
+### ✨ Features and improvements
+
+### 🐞 Bug fixes
+- **URLs shown in the console name the public port, not the admin one.** The console is served by the
+  admin listener, so every address it handed out named that listener — a TileJSON URL on `:8091`, a
+  `.torrent` on `:8091`, a style URL carrying both. Those are exactly the addresses that get pasted
+  into a style, a torrent client or another node, and none of them can reach the admin port: it binds
+  to localhost by default and serves nothing public even when it does not.
+
+  Only the port is corrected, and only when it is the admin one. The host stays whatever the request
+  arrived as, so a node behind a proxy still names itself correctly without anyone setting
+  `publicUrl` — and `publicUrl` still wins outright where it is set.
+
+## 0.45.3
+### ✨ Features and improvements
+
+### 🐞 Bug fixes
+- **The Copy and Open buttons on the Categories tab no longer sit off the edge of the page.** Table
+  cells are `nowrap`, which is right for the archive list where a wrapped number is worse than a wide
+  column — but a category cell holds a style URL and a paragraph explaining it, so nothing could
+  wrap and the table grew past the window, carrying the buttons at its end out of reach. The
+  endpoints table lays out fixed now: the label and button columns keep their width and the URL takes
+  what is left.
+
 ## 0.45.2
 ### ✨ Features and improvements
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pmtiles-swarm",
-  "version": "0.45.2",
+  "version": "0.46.0",
   "description": "BitTorrent distribution for PMTiles map archives: create torrents, watch folders, publish and subscribe to RSS feeds, and seed through qBittorrent or an embedded client",
   "type": "module",
   "main": "src/index.js",

package/src/api.js

@@ -317,8 +317,27 @@ export function createApp({
     // a torrent client handed the `torrent` link, another node syncing from
     // the feed -- silently gets something it cannot use.
     const configured = String(config.publicUrl ?? '').trim();
-    const base = configured || `${req.protocol}://${req.host}`;
-    return base.replace(/\/$/, '');
+    if (configured) return configured.replace(/\/$/, '');
+
+    // Never the admin port, whichever one the request came in on.
+    //
+    // The console lives on the admin listener, so every URL it showed named
+    // that listener — a TileJSON URL on :8091, a `.torrent` on :8091, a style
+    // URL carrying both. Those are the addresses people copy into a style, a
+    // torrent client or another node, and none of them can reach the admin
+    // port: it is bound to localhost by default and serves nothing public
+    // even when it is not. Half the addresses on a node pointing somewhere
+    // the other half do not is a confusion nobody should have to diagnose.
+    //
+    // The host is still whatever the request arrived as, so a node behind a
+    // proxy keeps naming itself correctly without anybody configuring
+    // `publicUrl`. Only the port is corrected, and only when it is the one
+    // port that is not for the public.
+    const host = onAdminPort(req)
+      ? String(req.host).replace(/:\d+$/, '') +
+        (config.port ? `:${config.port}` : '')
+      : req.host;
+    return `${req.protocol}://${host}`.replace(/\/$/, '');
   };
 
   /**

package/src/web/index.html

@@ -224,6 +224,15 @@
       }
       .tag.on { background: var(--accent); color: #fff; border-color: var(--accent); }
       #add-error { color: var(--bad); min-height: 1.2rem; }
+      /* Category endpoints. `th, td` is nowrap for the archive table, where a
+         wrapped number is worse than a wide one — but here the cell holds a
+         style URL and a paragraph explaining it, so nowrap pushed the table
+         wider than the window and took the Copy and Open buttons off the far
+         edge with it. Fixed layout keeps the two end columns where they are
+         and gives the middle whatever is left. */
+      #category-list table { table-layout: fixed; }
+      #category-list td { white-space: normal; }
+      #category-list code { word-break: break-all; }
       /* Separates what acts on one archive from what acts on all of them. */
       .bar-gap { flex: 1 1 1.5rem; min-width: 0.75rem; }
       .tabs {

package/src/web/public.html

@@ -118,6 +118,19 @@
         gap: 0.35rem 0.9rem;
         font-size: 0.9rem;
       }
+      /* A copy button reads as a link's sibling, not as a form control. */
+      .copy {
+        border: 1px solid var(--line);
+        border-radius: 999px;
+        background: none;
+        color: var(--muted);
+        font: inherit;
+        font-size: 0.74rem;
+        padding: 0 0.45rem;
+        cursor: pointer;
+      }
+      .copy:hover { color: var(--fg); border-color: var(--fg); }
+      .copy.done { border-color: currentColor; color: #3fb950; }
       .urls {
         margin-top: 0.7rem;
         display: grid;
@@ -390,30 +403,55 @@
             a.href = href;
             links.append(a);
           };
+          // What each endpoint is for decides how it is offered. A .torrent
+          // is a file to fetch, so it stays a link the browser downloads; a
+          // preview is a page to visit. The rest are addresses that belong
+          // somewhere else — a style, a torrent client, a feed reader — and
+          // following one here achieves nothing, so those copy.
+          const copy = (href, text) => {
+            const button = el('button', 'copy', text);
+            button.type = 'button';
+            const absolute = new URL(href, location.href).href;
+            // The URL either way: a browser on plain HTTP has no clipboard,
+            // and a tooltip somebody can select beats a button that does
+            // nothing.
+            button.title = absolute;
+            button.onclick = async () => {
+              try {
+                await navigator.clipboard.writeText(absolute);
+                button.classList.add('done');
+                button.textContent = 'copied';
+                setTimeout(() => {
+                  button.classList.remove('done');
+                  button.textContent = text;
+                }, 1200);
+              } catch {
+                button.textContent = 'copy failed — it is in the tooltip';
+              }
+            };
+            links.append(button);
+          };
+
           const ends = entry.endpoints;
-          if (ends.tileJson) add(ends.tileJson, 'TileJSON');
+          if (ends.tileJson) copy(ends.tileJson, 'TileJSON');
           // The newest build's preview, which is what "show me this
           // category" means.
           if (ends.preview) add(ends.preview, 'preview');
-          if (ends.torrent) add(ends.torrent, '.torrent');
-          if (ends.magnet) add(ends.magnet, 'magnet');
-          if (ends.feed) add(ends.feed, 'RSS');
-          card.append(links);
-
-          const urls = el('div', 'urls');
-          // The one worth copying: the TileJSON URL with the .torrent URL and
-          // the magnet in its fragment, which a plain client fetches over HTTP
-          // and a swarm-aware one joins directly.
-          if (ends.styleUrl) {
-            urls.append(
-              urlRow('style URL', new URL(ends.styleUrl, location.href).href),
-            );
-          } else if (ends.tileJson) {
-            urls.append(
-              urlRow('TileJSON', new URL(ends.tileJson, location.href).href),
-            );
+          // A file, so a link the browser saves rather than renders.
+          if (ends.torrent) {
+            const a = el('a', null, '.torrent');
+            a.href = ends.torrent;
+            a.download = '';
+            links.append(a);
           }
-          if (urls.childElementCount > 0) card.append(urls);
+          if (ends.magnet) copy(ends.magnet, 'magnet');
+          if (ends.feed) copy(ends.feed, 'RSS');
+          // The one worth copying and the one nobody wants to read: a TileJSON
+          // URL carrying the .torrent URL and a percent-encoded magnet in its
+          // fragment. Printed in full it was most of the card — several
+          // hundred characters of it — and it is never read, only pasted.
+          if (ends.styleUrl) copy(ends.styleUrl, 'style URL');
+          card.append(links);
 
           host.append(card);
         }