From c43f58dc3f41f881f4dcbeae63ca86159448696c Mon Sep 17 00:00:00 2001 From: Trygve Hemminghytt <274629477+trygvehhh@users.noreply.github.com> Date: Mon, 5 Oct 2026 21:46:20 +0200 Subject: [PATCH 1/9] feat: Kartverket ground heights for Statens vegvesen cameras The Vegvesen camera feed is 2D, so every camera shared one flat 150 m ground prior and mountain-pass cameras could sit hundreds of metres below the terrain where neither the point-height prior nor the ground snap resolves. Each camera now gets its terrain height from Kartverket's keyless point-height API (CC BY 4.0): ~840 of ~845 cameras. - 50 points per request, six in flight; points a full batch answers null for are retried once in batches of 10 (the API drops points from large batches). - Sea-floor depths (dybdekurver / Havflate) under bridges and quays count as the water surface, 0 m. - Heights are cached by position, so the 15-minute refresh only asks about new or moved cameras. A catalog load waits at most 12 s; a slower lookup keeps filling the cache for the next refresh. - CCTV_VEGVESEN_HEIGHTS=0 turns it off; Kartverket is credited beside Statens vegvesen in the attribution popover. Follow-up to #929. Co-Authored-By: Claude Opus 5.5 --- .env.example | 1 + CHANGELOG.md | 10 ++ DATA_SOURCES.md | 2 + docs/CURRENT-STATE.md | 2 +- server/providers/cctv/constants.js | 24 +++ server/providers/cctv/sources.js | 225 ++++++++++++++++++++++++++++- src/data/cctvVegvesen.test.mjs | 223 +++++++++++++++++++++++++++- src/data/dataCredits.js | 3 +- 8 files changed, 479 insertions(+), 11 deletions(-) diff --git a/.env.example b/.env.example index ee7620be9..99a3ff88b 100644 --- a/.env.example +++ b/.env.example @@ -195,6 +195,7 @@ AISSTREAM_API_KEY= # CCTV_VEGVESEN_ENABLED=1 # set 0 to disable the Statens vegvesen (Norway) road cameras # CCTV_VEGVESEN_MAX_SOURCES=900 # cap cameras loaded from Statens vegvesen (default loads all ~850 live; max 1000) # CCTV_VEGVESEN_VIDEO=1 # set 0 to show stills instead of live HLS for Statens vegvesen video cameras +# CCTV_VEGVESEN_HEIGHTS=1 # set 0 to skip Kartverket ground heights for Statens vegvesen cameras (flat prior instead) # CCTV_VEGVESEN_URL= # override the Statens vegvesen catalog URL # CCTV_MAX_SOURCES=4000 # catalog-wide ceiling, shared round-robin across packs # CCTV_PREFER_AUSTIN=1 # prioritize Austin cameras in the registry diff --git a/CHANGELOG.md b/CHANGELOG.md index b433f8c06..86cda068c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,6 +2,16 @@ ## [Unreleased] +- Give each Statens vegvesen camera its own ground height from Kartverket's + keyless point-height API (CC BY 4.0) instead of one flat 150 m prior, so + mountain-pass cameras no longer sit hundreds of metres below the terrain + where neither the point-height prior nor the ground snap resolves. About + 840 of ~845 cameras get a height; the rest keep the old prior. Bridge and + quay cameras over water use the sea surface, not the sea-floor depth the + API returns there. A catalog load waits at most 12 s for heights, and a + lookup still running then fills a cache the next refresh applies. + `CCTV_VEGVESEN_HEIGHTS=0` turns the lookup off. Follow-up to #929. + - Add Norway's road cameras from Statens vegvesen as a CCTV pack. The camera list is one keyless GeoJSON request to the agency's OGC API view of its DATEX 3.1 CCTV table, so no DATEX account is needed. Every working camera diff --git a/DATA_SOURCES.md b/DATA_SOURCES.md index 734e35a69..4959c1c75 100644 --- a/DATA_SOURCES.md +++ b/DATA_SOURCES.md @@ -55,6 +55,7 @@ How to read this: | **DriveBC** (Government of British Columbia) | CCTV camera catalog + frames, British Columbia highways | [Open Government Licence – British Columbia](https://www2.gov.bc.ca/gov/content/data/open-data/open-government-licence-bc) — attribution REQUIRED | "DriveBC. Contains information licensed under the Open Government Licence – British Columbia" | | **Open Calgary** (`data.calgary.ca`) | CCTV camera catalog + frames, Calgary | [Open Government Licence – City of Calgary](https://data.calgary.ca/stories/s/Open-Calgary-Terms-of-Use/u45n-7awa) — attribution REQUIRED | "Contains information licensed under the Open Government Licence – City of Calgary" | | **Statens vegvesen** (catalog `ogckart-sn1.atlas.vegvesen.no`, frames `kamera.atlas.vegvesen.no`, HLS `kamera.vegvesen.no`) | CCTV camera catalog + frames + live HLS video, Norway road cameras | [NLOD 2.0](https://data.norge.no/nlod/en/2.0) — attribution REQUIRED | "Contains data under the Norwegian licence for Open Government data (NLOD) distributed by Statens vegvesen" | +| **Kartverket** (`ws.geonorge.no/hoydedata/v1/punkt`) | Terrain heights (NN2000) used as the ground prior for Statens vegvesen cameras | [CC BY 4.0](https://creativecommons.org/licenses/by/4.0/) — attribution REQUIRED | "© Kartverket" | | **GBFS (Lyft / BCycle)** | Bikeshare availability | Per-feed (attribution-only) | Credit the operator (e.g. Austin BCycle) + its `license_url` | | **MBTA / MassDOT** (cdn.mbta.com) | Transit layer live vehicles and up to 15 minutes of recently observed positions, Greater Boston | MBTA / MassDOT open realtime vehicle data, used for the live view and up to fifteen minutes of recently observed positions; published for developer use and not against their use. | Attribution to MBTA / MassDOT (courtesy text) | | **CapMetro** (data.texas.gov) | Transit layer live vehicles, Austin TX | CapMetro open realtime vehicle data, used for the live view; published for developer use and not against their use. | Attribution to CapMetro (courtesy text) | @@ -90,6 +91,7 @@ The optional Local ADS-B layer and the broadcast-FM receiver use radio you recei - **Live Traffic NSW cameras.** The camera list is the keyless public `data.livetraffic.com/cameras/traffic-cam.json` feed (Transport for NSW open data, CC BY 4.0); frames are stills on `webcams.transport.nsw.gov.au`, and only that host is registered. That image host answers non-browser clients with an HTML placeholder, so the proxy identifies as a browser for that one host and as itself everywhere else. Frames are fetched at request time and never stored or redistributed. Disable with `CCTV_NSW_ENABLED=0`. - **Open Calgary traffic cameras.** Keyless. The camera list is the public Socrata dataset `k7p9-kppz` on `data.calgary.ca`; frames are stills on the city's own `trafficcam.calgary.ca` host, and only that host is registered. Most rows publish an `http://` frame URL and the host redirects those to HTTPS, so URLs are upgraded and pinned to that origin before registration. Frames are fetched at request time and never stored or redistributed. The dataset publishes no camera facing: the `quadrant` field and the quadrant suffix on each camera name are Calgary's address grid, not a bearing, so headings use the shared id-hash fallback at low confidence and are corrected with the in-app calibration gizmo. The licence requires the attribution statement above. Disable with `CCTV_CALGARY_ENABLED=0`. - **Statens vegvesen road cameras.** Keyless. The camera list is one GeoJSON request to the OGC API Features view of the official DATEX 3.1 CCTV site table (`ogckart-sn1.atlas.vegvesen.no/ogc/features/v1/collections/datex_3_1:CctvSimple/items`), so no DATEX account is needed; frames are stills on `kamera.atlas.vegvesen.no/api/images/`, and only that exact path per camera is registered. Cameras the feed reports as faulty are dropped. About 135 working cameras also publish live HLS (`kamera.vegvesen.no/public//manifest.m3u8`, MPEG-TS); those play as live video through the bounded in-memory HLS relay, with the still as the snapshot fallback, and only that exact manifest path per camera is registered. `CCTV_VEGVESEN_VIDEO=0` keeps every camera on stills. `orientationDescription` names the place a camera faces, not a bearing, so it goes into the label and headings use the id-hash fallback at low confidence. By default every live camera loads (~850); `CCTV_VEGVESEN_MAX_SOURCES` (8–1000) lowers the cap, keeping the cameras nearest Oslo, Bergen, Stavanger, Trondheim, Kristiansand, Bodø and Tromsø and `CCTV_VEGVESEN_ENABLED=0` turns the pack off. Frames are fetched at request time and never stored or redistributed. The NLOD attribution is registered in the Data attribution popover. +- **Kartverket ground heights for Statens vegvesen cameras.** The camera feed's points are 2D, so the loader asks Kartverket's keyless point-height API (`ws.geonorge.no/hoydedata/v1/punkt`, CC BY 4.0) for the terrain height at each camera, 50 points per request with up to six in flight (about 17 requests, ~5 s, for the whole country). Points a full batch answers `null` for are retried once in batches of 10, since the API sometimes drops points from large batches; points still without data keep the flat 150 m prior. Over water the API returns the sea-floor depth (`dybdekurver` / `Havflate`), which is replaced by 0 m, the water surface. Heights are cached in memory by position, so the 15-minute catalog refresh only asks about new or moved cameras. A catalog load waits at most 12 s; a slower lookup keeps filling the cache for the next refresh. Only coordinates are sent. `CCTV_VEGVESEN_HEIGHTS=0` turns the lookup off. The Kartverket credit is shown beside the Statens vegvesen entry in the Data attribution popover. - **Camera frame content.** Public camera frames can contain people, vehicles and license plates, and the camera layer does nothing with them beyond putting them on the map. A successful upstream response is relayed as the provider served it: nothing in the camera pipeline blurs, redacts, enhances or restores it, and nothing there runs face, plate or object detection or otherwise recognises or classifies what is in the picture. (The ALPR layer described below maps where plate-reader cameras are, from locations contributors tagged in OpenStreetMap; it reads nothing from any camera frame.) When an upstream has no frame for a camera, what you see instead is a Google Street View still or the app's own placeholder card; the panel's source badge names which of them you are looking at, and a refresh that fails after a frame has arrived leaves the last good picture on screen rather than replacing it. The browser resamples each frame for display: onto a fixed 1920x1080 surface for the camera's projection plane in the scene, and onto a 192x108 canvas for the small ambient card. That scaling is the only change made to the picture, and the only pixels the camera layer reads back are a 64x36 downscale hashed to tell whether the picture has changed. Live streams are piped through; the still path reads the image into memory to pass it on — TxDOT wraps its JPEG in a JSON envelope, which is decoded there — and no frame is written to disk (the disk cache, `.gev-cache/`, holds feed JSON for other layers and never camera imagery). Outside that pipeline, the voice assistant is the one feature that sends imagery anywhere: when it answers a question about what is in view it may attach a downscaled screenshot of the rendered scene — including any camera plane on screen — to the request it sends to the voice model. Frames are fetched only from server-registered camera URLs — a client cannot hand the proxy a URL (see [SECURITY.md](SECURITY.md)). What the app does stays inside the line README draws: no named-person search, no face recognition, no tracking individuals. - **Transit (GTFS-Realtime).** The proxy requests only registered operator feeds, validates redirects, bounds responses, and uses conditional requests, short snapshot caching and failure backoff. All seven registered regions are live-enabled; each can be disabled independently. Vehicles use delayed playback between reports, with report age and waiting states shown separately. Session-local selected trails are available for every feed. MBTA additionally retains up to 15 minutes of recently observed positions in the running proxy, so available history can survive a browser reload; server restart clears it. History reads never fetch upstream, and other feeds have no proxy retention. Nothing is archived to disk or collected in the background. Operator text credits are included as a courtesy and do not imply endorsement. Heights are aligned to work with Google 3D tiles; subway markers are shown at street level because these feeds do not supply depth. - **Radio Browser.** `/api/radio/stations` discovers official API mirrors, makes bounded and coalesced healthy/geolocated HTTPS-station queries, caches the normalized public-domain directory for 45 minutes, and may serve the last good catalog for up to seven days during an outage. Refreshes must meet minimum accepted-query and station coverage before replacing a warm catalog; schema-valid responses whose rows all fail the product's health policy do not count as successful queries. A usable partial cold catalog is explicitly `DEGRADED`, and malformed or empty successful payloads are rejected atomically. Every directory and click-count request rejects redirects, validates all resolved addresses as globally routable (including reserved/documentation IPv4 and special/non-global IPv6 exclusions), and pins the TLS connection to a validated address. Only MP3/AAC non-HLS directory rows with public HTTPS stream targets are returned; favicons are intentionally omitted. Pressing play connects one browser audio element directly to the selected broadcaster and calls the directory's click counter through known-ID-only `POST /api/radio/click/:uuid`. GEV never proxies, caches, records, bundles, or redistributes audio. Radio Browser supplies station-level tags, not dependable current-song or upcoming-program metadata, so Radio filtering never claims either. Direct playback exposes the listener's IP address to the broadcaster, whose own stream terms apply. diff --git a/docs/CURRENT-STATE.md b/docs/CURRENT-STATE.md index d4fe022a8..c5bf9df9e 100644 --- a/docs/CURRENT-STATE.md +++ b/docs/CURRENT-STATE.md @@ -2919,7 +2919,7 @@ its criteria cannot be silently ignored. | Satellites | CelesTrak | `src/data/satellites.js` | `/api/celestrak` | 120s | | Space Missions (30d) | Launch Library 2 + CelesTrak | `src/data/rocketLaunches.js` | `/api/launches` + `/api/celestrak/active` | 5 min | | Traffic | Selectable TomTom / OpenStreetMap / Hybrid roads; optional TomTom flow (BYOK) | `src/data/traffic.js` | browser-direct OpenFreeMap tiles; `/api/tomtom` for flow | viewport-driven; capped tile cache | -| CCTV | Austin + Caltrans (CA) + TfL London + Ontario 511 + Fintraffic (FI) + DriveBC (BC) + TxDOT (TX) + Estonia (Tallinn, Tarktee) + Live Traffic NSW + Open Calgary Open Data + Statens vegvesen (NO, stills + live HLS) + Street View fallback | `src/data/cctv.js` | `/api/cctv` | 10s (active) | +| CCTV | Austin + Caltrans (CA) + TfL London + Ontario 511 + Fintraffic (FI) + DriveBC (BC) + TxDOT (TX) + Estonia (Tallinn, Tarktee) + Live Traffic NSW + Open Calgary Open Data + Statens vegvesen (NO, stills + live HLS, Kartverket ground heights) + Street View fallback | `src/data/cctv.js` | `/api/cctv` | 10s (active) | | Radio | Radio Browser (public-domain station directory) | `src/data/radio.js` | `/api/radio/stations`, `/api/radio/click/:uuid` | 45 min directory refresh | | Transit 🚌 | Operator GTFS-Realtime VehiclePositions (7 keyless regions, `src/data/transitFeeds.js`) | `src/layers/transit/` via `src/app/layers/transit.js` | `/api/transit` | 15s (poll + delayed playback) | | Bikeshare 🚲 | GBFS (Lyft + BCycle) | `src/data/bikeshare.js` | `/api/gbfs` | 60s | diff --git a/server/providers/cctv/constants.js b/server/providers/cctv/constants.js index 6298e86a0..7609a3fc5 100644 --- a/server/providers/cctv/constants.js +++ b/server/providers/cctv/constants.js @@ -267,6 +267,30 @@ export const NORWAY_ANCHORS = [ { lat: 69.6492, lon: 18.9553 }, // Tromsø ]; +/** Kartverket's keyless point-height API (CC BY 4.0): terrain height above + * sea level (NN2000) for up to 50 points per request, used as the Vegvesen + * cameras' ground prior because the camera feed's points are 2D. */ +export const KARTVERKET_HEIGHT_URL = + 'https://ws.geonorge.no/hoydedata/v1/punkt'; +export const KARTVERKET_MAX_POINTS = 50; +/** Batch size for the one retry of points a full batch answered null for. */ +export const KARTVERKET_RETRY_POINTS = 10; +/** Batches in flight at once. The whole country is ~17 batches of ~2 s each + * (spread-out points are slow to resolve), so six keep a cold lookup ~6 s. */ +export const KARTVERKET_CONCURRENCY = 6; +/** Deadline for one complete lookup, shared by every batch. */ +export const KARTVERKET_LOOKUP_TIMEOUT_MS = 30 * 1000; +/** How long a catalog load waits for heights. Kept under the per-provider + * fetch timeout so this pack is never the catalog's slowest lane; a lookup + * still running after this keeps filling the cache, and the next refresh + * applies the rest. At app start, with every pack loading at once, a cold + * lookup takes ~10 s. */ +export const KARTVERKET_CATALOG_WAIT_MS = 12 * 1000; +export const KARTVERKET_MAX_RESPONSE_BYTES = 256 * 1024; +/** Plausible Norwegian terrain: below sea level only at the shore, and no road + * camera sits above ~1,500 m. Anything outside is treated as no answer. */ +export const KARTVERKET_HEIGHT_RANGE_M = Object.freeze({ min: -20, max: 2500 }); + /** DelDOT CCTV: one keyless statewide JSON catalog; live video via RTMP-over-HTTP (rtmpt:80). */ export const DELDOT_CCTV_URL = 'https://tmc.deldot.gov/json/videocamera.json'; export const DEFAULT_DELDOT_MAX_SOURCES = 300; diff --git a/server/providers/cctv/sources.js b/server/providers/cctv/sources.js index 7ecd3434d..281ddde85 100644 --- a/server/providers/cctv/sources.js +++ b/server/providers/cctv/sources.js @@ -65,6 +65,14 @@ import { DEFAULT_VEGVESEN_MAX_SOURCES, VEGVESEN_MAX_CATALOG_BYTES, NORWAY_ANCHORS, + KARTVERKET_HEIGHT_URL, + KARTVERKET_MAX_POINTS, + KARTVERKET_RETRY_POINTS, + KARTVERKET_CONCURRENCY, + KARTVERKET_LOOKUP_TIMEOUT_MS, + KARTVERKET_CATALOG_WAIT_MS, + KARTVERKET_MAX_RESPONSE_BYTES, + KARTVERKET_HEIGHT_RANGE_M, } from './constants.js'; import { toFiniteNumber, @@ -1779,14 +1787,11 @@ export function vegvesenCameraToSource(feature) { fovDeg: 44, rangeM: 145, mountHeightM: 8, - // KNOWN LIMITATION: the feed's coordinates are 2D, so this is one flat - // prior for the whole country, while cameras run from sea level to - // mountain passes near 1,000 m (Haukelifjell, Sjonfjellet, Rugeldalen). - // The point-height prior and the client's one-shot ground snap correct it - // where they resolve; on a stack where neither does (no 3D tiles), a pass - // camera stays hundreds of metres below the terrain, the same risk the - // Caltrans pack documents. A per-camera height (e.g. Kartverket's keyless - // point-height API) would remove it. + // The feed's coordinates are 2D. The loader replaces this flat prior with + // Kartverket's terrain height for each camera (applyVegvesenGroundHeights); + // it only survives when that lookup fails, and then a mountain-pass camera + // can sit below the terrain on a stack without 3D tiles, the same risk the + // Caltrans pack documents. groundElevationM: 150, feedType: videoUrl ? 'hls' : 'image', url: videoUrl || imageUrl, @@ -1851,6 +1856,7 @@ export async function loadVegvesenSourcesFromOpenData() { ? Math.max(8, Math.min(1000, Math.floor(maxRaw))) : DEFAULT_VEGVESEN_MAX_SOURCES; const prioritized = prioritizeSources(cameras, maxCount, NORWAY_ANCHORS); + await applyVegvesenGroundHeights(prioritized); console.log( `[CCTV] Loaded Vegvesen camera sources: ${cameras.length} (using nearest ${prioritized.length})`, ); @@ -1863,3 +1869,206 @@ export async function loadVegvesenSourcesFromOpenData() { return []; } } + +/** + * Terrain heights already resolved, keyed by rounded position. Camera + * positions rarely change, so after the first catalog load a 15-minute refresh + * asks Kartverket only for new or moved cameras. Bounded well above the ~900 + * cameras the pack can load. + */ +const kartverketHeightCache = new Map(); +const KARTVERKET_CACHE_LIMIT = 5000; +/** The lookup in flight, so overlapping catalog loads never start a second. */ +let kartverketPending = null; + +/** Cache key for one position, at the ~0.1 m precision the request sends. */ +const kartverketKey = (lon, lat) => `${lon.toFixed(6)},${lat.toFixed(6)}`; + +/** Forget cached heights and any lookup in flight. Tests only. */ +export function clearKartverketHeightCache() { + kartverketHeightCache.clear(); + kartverketPending = null; +} + +/** + * Terrain heights above sea level (NN2000) from Kartverket's keyless + * point-height API, in batches of up to 50 points. Answers are matched back by + * the coordinates the API echoes, not by position in the list. Each answer is + * written to `into` as its batch lands, so a caller that stops waiting still + * keeps what has arrived. One deadline covers every request. + * + * Two quirks of the live API are handled here: + * - A large batch sometimes answers `z: null` for a point it resolves when + * asked again (Valdresflye: null in a 50-point batch, 1,390.7 m alone), so + * every point still missing after the first pass is retried once in batches + * of KARTVERKET_RETRY_POINTS. A point still null then has no data. + * - Over water `/punkt` returns the depth of the sea floor (`dybdekurver`, + * terrain `Havflate`; −547 m under Nordhordlandsbrua). A camera on a bridge + * or quay stands over the water surface, so those answers count as 0 m. + * + * @param {Array<{lon:number, lat:number}>} points + * @param {{into?: Map, timeoutMs?: number}} [options] + * @returns {Promise>} kartverketKey(lon, lat) → metres. + */ +export async function fetchKartverketHeights( + points, + { into = new Map(), timeoutMs = KARTVERKET_LOOKUP_TIMEOUT_MS } = {}, +) { + const unique = Array.from( + new Map( + (points || []) + .filter((p) => isLikelyNorwayCoordinate(p?.lat, p?.lon)) + .map((p) => [kartverketKey(p.lon, p.lat), p]), + ).values(), + ); + if (!unique.length) return into; + const signal = AbortSignal.timeout(timeoutMs); + const fetchBatch = async (batch) => { + const url = new URL(KARTVERKET_HEIGHT_URL); + url.searchParams.set( + 'punkter', + JSON.stringify( + batch.map((p) => [Number(p.lon.toFixed(6)), Number(p.lat.toFixed(6))]), + ), + ); + url.searchParams.set('koordsys', '4258'); + const resp = await fetch(url, { + headers: { Accept: 'application/json' }, + redirect: 'error', + signal, + }); + if (!resp.ok) { + try { + await resp.body?.cancel(); + } catch { + /* no-op */ + } + throw new Error(`HTTP ${resp.status}`); + } + const payload = await readResponseJsonCapped( + resp, + KARTVERKET_MAX_RESPONSE_BYTES, + ); + for (const point of Array.isArray(payload?.punkter) + ? payload.punkter + : []) { + // `z: null` means no answer for that point; Number(null) would read 0. + if (point?.z === null || point?.z === undefined || point?.z === '') + continue; + const x = toFiniteNumber(point?.x); + const y = toFiniteNumber(point?.y); + let z = toFiniteNumber(point?.z); + if (![x, y, z].every(Number.isFinite)) continue; + if (point?.datakilde === 'dybdekurver' || point?.terreng === 'Havflate') + z = 0; + if ( + z < KARTVERKET_HEIGHT_RANGE_M.min || + z > KARTVERKET_HEIGHT_RANGE_M.max + ) + continue; + into.set(kartverketKey(x, y), z); + } + }; + let failed = 0; + const runPass = async (list, size) => { + const batches = []; + for (let i = 0; i < list.length; i += size) { + batches.push(list.slice(i, i + size)); + } + let next = 0; + const worker = async () => { + while (next < batches.length && !signal.aborted) { + const batch = batches[next++]; + try { + await fetchBatch(batch); + } catch (error) { + failed += 1; + if (failed === 1) { + console.warn( + '[CCTV] Kartverket height lookup failed:', + error?.message || error, + ); + } + } + } + }; + await Promise.all( + Array.from( + { length: Math.min(KARTVERKET_CONCURRENCY, batches.length) }, + worker, + ), + ); + }; + await runPass(unique, KARTVERKET_MAX_POINTS); + const missing = unique.filter((p) => !into.has(kartverketKey(p.lon, p.lat))); + if (missing.length && !signal.aborted) { + await runPass(missing, KARTVERKET_RETRY_POINTS); + } + return into; +} + +/** + * Give each Vegvesen camera its own terrain height as the ground prior, in + * place. Waits at most `waitMs` for heights it does not have cached; a lookup + * still running then keeps filling the cache for the next catalog refresh. + * Cameras without a height keep the flat prior. Never throws: a failed lookup + * only means the old behavior. CCTV_VEGVESEN_HEIGHTS=0 turns it off. + * + * @param {Array} cameras - Sources from vegvesenCameraToSource(). + * @param {{waitMs?: number}} [options] + * @returns {Promise} How many cameras got a terrain height. + */ +export async function applyVegvesenGroundHeights( + cameras, + { waitMs = KARTVERKET_CATALOG_WAIT_MS } = {}, +) { + if (String(process.env.CCTV_VEGVESEN_HEIGHTS || '1').trim() === '0') return 0; + const list = Array.isArray(cameras) ? cameras : []; + const missing = list.filter( + (c) => !kartverketHeightCache.has(kartverketKey(c.lon, c.lat)), + ); + if (missing.length) { + if (!kartverketPending) { + if ( + kartverketHeightCache.size + missing.length > + KARTVERKET_CACHE_LIMIT + ) { + kartverketHeightCache.clear(); + } + kartverketPending = fetchKartverketHeights(missing, { + into: kartverketHeightCache, + }) + .catch((error) => { + console.warn( + '[CCTV] Kartverket height lookup error:', + error?.message || error, + ); + }) + .finally(() => { + kartverketPending = null; + }); + } + let timer; + await Promise.race([ + kartverketPending, + new Promise((resolve) => { + timer = setTimeout(resolve, waitMs); + timer.unref?.(); + }), + ]); + clearTimeout(timer); + } + let applied = 0; + for (const camera of list) { + const z = kartverketHeightCache.get(kartverketKey(camera.lon, camera.lat)); + if (!Number.isFinite(z)) continue; + camera.groundElevationM = Math.round(z * 10) / 10; + applied += 1; + } + if (list.length) { + console.log( + `[CCTV] Kartverket ground heights: ${applied}/${list.length} Vegvesen cameras`, + ); + } + return applied; +} diff --git a/src/data/cctvVegvesen.test.mjs b/src/data/cctvVegvesen.test.mjs index 457fecbda..ffb25aec3 100644 --- a/src/data/cctvVegvesen.test.mjs +++ b/src/data/cctvVegvesen.test.mjs @@ -2,11 +2,17 @@ import { test } from 'node:test'; import assert from 'node:assert/strict'; import { + applyVegvesenGroundHeights, + clearKartverketHeightCache, + fetchKartverketHeights, loadVegvesenSourcesFromOpenData, vegvesenCameraToSource, } from '../../server/providers/cctv/sources.js'; import { DEFAULT_VEGVESEN_CCTV_URL, + KARTVERKET_HEIGHT_URL, + KARTVERKET_MAX_POINTS, + KARTVERKET_RETRY_POINTS, VEGVESEN_IMAGE_ORIGIN, } from '../../server/providers/cctv/constants.js'; import { createCctvCatalog } from '../../server/providers/cctv/catalog.js'; @@ -104,6 +110,8 @@ test('faulty cameras, off-host frames and bad geometry are dropped', () => { }); test('the loader dedupes, and failures degrade to an empty pack', async (t) => { + clearKartverketHeightCache(); + t.after(clearKartverketHeightCache); t.mock.method(console, 'log', () => {}); t.mock.method(console, 'warn', () => {}); const requested = []; @@ -127,7 +135,11 @@ test('the loader dedupes, and failures degrade to an empty pack', async (t) => { }); }); const cameras = await loadVegvesenSourcesFromOpenData(); - assert.deepEqual(requested, [DEFAULT_VEGVESEN_CCTV_URL]); + // The catalog first; the rest is the Kartverket height lookup. + assert.equal(requested[0], DEFAULT_VEGVESEN_CCTV_URL); + assert.ok( + requested.slice(1).every((url) => url.startsWith(KARTVERKET_HEIGHT_URL)), + ); // Nearest-to-anchor first: Oslo leads. assert.deepEqual( cameras.map((c) => c.id), @@ -200,3 +212,212 @@ test('CCTV_VEGVESEN_ENABLED=0 keeps the lane from being loaded', async (t) => { ); }); }); + +/** Answer a Kartverket request the way the live API does: echo every point, + * with the height `heightFor(x, y)` returns (null = outside coverage). */ +const kartverketResponse = (url, heightFor) => { + const points = JSON.parse(new URL(url).searchParams.get('punkter')); + return Response.json({ + koordsys: 4258, + punkter: points.map(([x, y]) => ({ x, y, z: heightFor(x, y) })), + }); +}; + +/** Cameras at distinct Norwegian positions. */ +const cameraAt = (i) => ({ + id: `no-vegvesen-${i}`, + lat: 60 + i * 0.001, + lon: 7 + i * 0.001, + groundElevationM: 150, +}); + +const withHeightsEnv = (t) => { + const saved = process.env.CCTV_VEGVESEN_HEIGHTS; + delete process.env.CCTV_VEGVESEN_HEIGHTS; + clearKartverketHeightCache(); + t.after(() => { + clearKartverketHeightCache(); + if (saved === undefined) delete process.env.CCTV_VEGVESEN_HEIGHTS; + else process.env.CCTV_VEGVESEN_HEIGHTS = saved; + }); + t.mock.method(console, 'log', () => {}); + t.mock.method(console, 'warn', () => {}); +}; + +test('Kartverket heights come in batches of 50, matched by echoed coordinates', async (t) => { + withHeightsEnv(t); + const requested = []; + t.mock.method(globalThis, 'fetch', async (url) => { + requested.push(String(url)); + assert.ok(String(url).startsWith(KARTVERKET_HEIGHT_URL)); + // Reply in reverse order: matching must not depend on list position. + const response = kartverketResponse(url, (x) => + x > 7.05 ? null : Math.round((x - 7) * 10000), + ); + const body = await response.json(); + body.punkter.reverse(); + return Response.json(body); + }); + const points = Array.from({ length: 120 }, (_, i) => cameraAt(i)); + const heights = await fetchKartverketHeights(points); + const sizes = requested.map( + (url) => JSON.parse(new URL(url).searchParams.get('punkter')).length, + ); + // Three full batches, then one retry pass over the 69 points with no height. + assert.deepEqual(sizes.slice(0, 3), [KARTVERKET_MAX_POINTS, 50, 20]); + assert.equal( + sizes.slice(3).reduce((a, b) => a + b, 0), + 69, + ); + assert.ok(sizes.slice(3).every((n) => n <= KARTVERKET_RETRY_POINTS)); + for (const url of requested) { + assert.equal(new URL(url).searchParams.get('koordsys'), '4258'); + } + // Points past x = 7.05 have no height (outside coverage) and are left out. + assert.equal(heights.size, 51); + assert.equal(heights.get('7.010000,60.010000'), 100); +}); + +test('points a full batch answers null for are retried once in small batches', async (t) => { + withHeightsEnv(t); + const sizes = []; + t.mock.method(globalThis, 'fetch', async (url) => { + const points = JSON.parse(new URL(url).searchParams.get('punkter')); + sizes.push(points.length); + // A full batch drops the first two points; a small batch answers them, + // except the one point that has no data at all. + const full = points.length > KARTVERKET_RETRY_POINTS; + return kartverketResponse(url, (x) => { + if (Math.abs(x - 7.002) < 1e-9) return null; + if (full && x < 7.002) return null; + return 500; + }); + }); + const heights = await fetchKartverketHeights( + Array.from({ length: 60 }, (_, i) => cameraAt(i)), + ); + assert.deepEqual(sizes, [50, 10, 3]); + assert.equal(heights.size, 59); + assert.equal(heights.get('7.000000,60.000000'), 500); + assert.equal(heights.has('7.002000,60.002000'), false); +}); + +test('a sea-floor depth counts as the water surface', async (t) => { + withHeightsEnv(t); + t.mock.method(globalThis, 'fetch', async (url) => { + const points = JSON.parse(new URL(url).searchParams.get('punkter')); + return Response.json({ + koordsys: 4258, + punkter: points.map(([x, y], i) => + i === 0 + ? { x, y, z: -547.1, datakilde: 'dybdekurver', terreng: 'Havflate' } + : { x, y, z: -0.2, datakilde: 'dtm1', terreng: 'ÅpentOmråde' }, + ), + }); + }); + const heights = await fetchKartverketHeights([cameraAt(0), cameraAt(1)]); + // Under a bridge: the surface, not the sea floor. A quay just below the + // datum on land data stays as measured. + assert.equal(heights.get('7.000000,60.000000'), 0); + assert.equal(heights.get('7.001000,60.001000'), -0.2); +}); + +test('implausible heights and points outside Norway are ignored', async (t) => { + withHeightsEnv(t); + const requested = []; + t.mock.method(globalThis, 'fetch', async (url) => { + requested.push(String(url)); + return kartverketResponse(url, () => 99999); + }); + const heights = await fetchKartverketHeights([ + cameraAt(1), + { lat: 55.68, lon: 12.57 }, // Copenhagen: never sent + ]); + assert.equal(heights.size, 0); + // The first pass and its one retry, each carrying only the Norway point. + assert.equal(requested.length, 2); + for (const url of requested) { + assert.equal( + JSON.parse(new URL(url).searchParams.get('punkter')).length, + 1, + ); + } +}); + +test('cameras get their terrain height, and a refresh reuses the cache', async (t) => { + withHeightsEnv(t); + let calls = 0; + t.mock.method(globalThis, 'fetch', async (url) => { + calls += 1; + return kartverketResponse(url, (x) => (x < 7.002 ? 1043.26 : null)); + }); + const cameras = [cameraAt(0), cameraAt(1), cameraAt(5)]; + assert.equal(await applyVegvesenGroundHeights(cameras), 2); + assert.deepEqual( + cameras.map((c) => c.groundElevationM), + [1043.3, 1043.3, 150], + ); + // One full pass plus one retry for the point with no height. + assert.equal(calls, 2); + // Same positions on the next catalog refresh: answered from the cache. + const again = [cameraAt(0), cameraAt(1)]; + assert.equal(await applyVegvesenGroundHeights(again), 2); + assert.equal(calls, 2); +}); + +test('a failed or disabled lookup keeps the flat prior', async (t) => { + withHeightsEnv(t); + const fetchMock = t.mock.method(globalThis, 'fetch', async () => { + throw new Error('offline'); + }); + const cameras = [cameraAt(0)]; + assert.equal(await applyVegvesenGroundHeights(cameras), 0); + assert.equal(cameras[0].groundElevationM, 150); + + fetchMock.mock.mockImplementation( + async () => new Response('nope', { status: 503 }), + ); + assert.equal(await applyVegvesenGroundHeights(cameras), 0); + assert.equal(cameras[0].groundElevationM, 150); + + process.env.CCTV_VEGVESEN_HEIGHTS = '0'; + fetchMock.mock.resetCalls(); + assert.equal(await applyVegvesenGroundHeights(cameras), 0); + assert.equal(fetchMock.mock.callCount(), 0); +}); + +test('a slow lookup does not hold the catalog; the next refresh applies it', async (t) => { + withHeightsEnv(t); + let release; + const gate = new Promise((resolve) => { + release = resolve; + }); + t.mock.method(globalThis, 'fetch', async (url) => { + await gate; + return kartverketResponse(url, () => 812); + }); + const first = [cameraAt(0)]; + assert.equal(await applyVegvesenGroundHeights(first, { waitMs: 20 }), 0); + assert.equal(first[0].groundElevationM, 150); + release(); + // The lookup kept running and filled the cache for the next refresh. + await new Promise((resolve) => setTimeout(resolve, 20)); + const next = [cameraAt(0)]; + assert.equal(await applyVegvesenGroundHeights(next, { waitMs: 20 }), 1); + assert.equal(next[0].groundElevationM, 812); +}); + +test('the loader applies Kartverket heights to the cameras it returns', async (t) => { + withHeightsEnv(t); + t.mock.method(globalThis, 'fetch', async (url) => { + if (String(url).startsWith(KARTVERKET_HEIGHT_URL)) { + return kartverketResponse(url, () => 987.6); + } + return Response.json({ type: 'FeatureCollection', features: [feature()] }); + }); + const cameras = await loadVegvesenSourcesFromOpenData(); + assert.deepEqual( + cameras.map((c) => [c.id, c.groundElevationM]), + [['no-vegvesen-3000063_1', 987.6]], + ); +}); diff --git a/src/data/dataCredits.js b/src/data/dataCredits.js index 2967f0a2b..8ad585eca 100644 --- a/src/data/dataCredits.js +++ b/src/data/dataCredits.js @@ -245,7 +245,8 @@ export const DATA_CREDITS = [ html: 'Road cameras (Norway): contains data under the ' + 'Norwegian licence for Open Government data (NLOD) ' + - 'distributed by Statens vegvesen', + 'distributed by Statens vegvesen · camera ground heights © ' + + 'Kartverket (CC BY 4.0)', }, { key: 'gbfs', From 34d53edc9188c996259b84e8c62ebb4d3697af08 Mon Sep 17 00:00:00 2001 From: Trygve Hemminghytt <274629477+trygvehhh@users.noreply.github.com> Date: Tue, 6 Oct 2026 19:21:27 +0200 Subject: [PATCH 2/9] fix: address review on Kartverket ground heights - Send 20 points per request, not 50: above ~20 Kartverket answers the extra points from contour lines (hoydekurver) or not at all. At 20, nearly every camera resolves from dtm1, so the null-retry pass is gone. Eight requests in flight keep a cold lookup ~6 s. - Cache misses: positions with no height are not asked again for 6 h. - Transport failures: the first 429, 5xx or network error stops the lookup with no retries, and no lookup starts until the backoff ends (Retry-After when sent, else 10 min; bounded to 1 min-1 h). - Budget the 12 s wait from the start of the loader, catalog fetch included. - Record answers only for the points each request asked about. - Send the CCTV User-Agent; link CC BY 4.0 in the credit. - Docs: bridge cameras sit at the water surface, below the deck. - Tests: batching, 8-in-flight cap, request count on failure, the shared deadline, Retry-After bounds, null TTL, backoff, the -20 m bound and each water marker on its own. Co-Authored-By: Claude Opus 5.5 --- CHANGELOG.md | 15 +- DATA_SOURCES.md | 2 +- server/providers/cctv/constants.js | 39 ++-- server/providers/cctv/sources.js | 267 +++++++++++++++++---------- src/data/cctvVegvesen.test.mjs | 282 ++++++++++++++++++++--------- src/data/dataCredits.js | 3 +- 6 files changed, 407 insertions(+), 201 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 86cda068c..152279ea0 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,11 +6,16 @@ keyless point-height API (CC BY 4.0) instead of one flat 150 m prior, so mountain-pass cameras no longer sit hundreds of metres below the terrain where neither the point-height prior nor the ground snap resolves. About - 840 of ~845 cameras get a height; the rest keep the old prior. Bridge and - quay cameras over water use the sea surface, not the sea-floor depth the - API returns there. A catalog load waits at most 12 s for heights, and a - lookup still running then fills a cache the next refresh applies. - `CCTV_VEGVESEN_HEIGHTS=0` turns the lookup off. Follow-up to #929. + 850 of ~855 cameras get a height; the rest keep the old prior. Requests + carry 20 points, the most the API answers from its 1 m terrain model, eight + at a time (~6 s cold). Cameras over water use the water surface rather than + the sea-floor depth the API returns there; for bridge cameras that is still + below the deck. Heights are cached for good and misses for 6 h, so a + refresh only asks about new or moved cameras. After a 429, a 5xx or a + network failure the lookup stops and backs off (Retry-After, else 10 min). + The lane waits at most 12 s from its start; a slower lookup fills the cache + for the next refresh. `CCTV_VEGVESEN_HEIGHTS=0` turns it off. Follow-up to + #929. - Add Norway's road cameras from Statens vegvesen as a CCTV pack. The camera list is one keyless GeoJSON request to the agency's OGC API view of its diff --git a/DATA_SOURCES.md b/DATA_SOURCES.md index 4959c1c75..fb53776c0 100644 --- a/DATA_SOURCES.md +++ b/DATA_SOURCES.md @@ -91,7 +91,7 @@ The optional Local ADS-B layer and the broadcast-FM receiver use radio you recei - **Live Traffic NSW cameras.** The camera list is the keyless public `data.livetraffic.com/cameras/traffic-cam.json` feed (Transport for NSW open data, CC BY 4.0); frames are stills on `webcams.transport.nsw.gov.au`, and only that host is registered. That image host answers non-browser clients with an HTML placeholder, so the proxy identifies as a browser for that one host and as itself everywhere else. Frames are fetched at request time and never stored or redistributed. Disable with `CCTV_NSW_ENABLED=0`. - **Open Calgary traffic cameras.** Keyless. The camera list is the public Socrata dataset `k7p9-kppz` on `data.calgary.ca`; frames are stills on the city's own `trafficcam.calgary.ca` host, and only that host is registered. Most rows publish an `http://` frame URL and the host redirects those to HTTPS, so URLs are upgraded and pinned to that origin before registration. Frames are fetched at request time and never stored or redistributed. The dataset publishes no camera facing: the `quadrant` field and the quadrant suffix on each camera name are Calgary's address grid, not a bearing, so headings use the shared id-hash fallback at low confidence and are corrected with the in-app calibration gizmo. The licence requires the attribution statement above. Disable with `CCTV_CALGARY_ENABLED=0`. - **Statens vegvesen road cameras.** Keyless. The camera list is one GeoJSON request to the OGC API Features view of the official DATEX 3.1 CCTV site table (`ogckart-sn1.atlas.vegvesen.no/ogc/features/v1/collections/datex_3_1:CctvSimple/items`), so no DATEX account is needed; frames are stills on `kamera.atlas.vegvesen.no/api/images/`, and only that exact path per camera is registered. Cameras the feed reports as faulty are dropped. About 135 working cameras also publish live HLS (`kamera.vegvesen.no/public//manifest.m3u8`, MPEG-TS); those play as live video through the bounded in-memory HLS relay, with the still as the snapshot fallback, and only that exact manifest path per camera is registered. `CCTV_VEGVESEN_VIDEO=0` keeps every camera on stills. `orientationDescription` names the place a camera faces, not a bearing, so it goes into the label and headings use the id-hash fallback at low confidence. By default every live camera loads (~850); `CCTV_VEGVESEN_MAX_SOURCES` (8–1000) lowers the cap, keeping the cameras nearest Oslo, Bergen, Stavanger, Trondheim, Kristiansand, Bodø and Tromsø and `CCTV_VEGVESEN_ENABLED=0` turns the pack off. Frames are fetched at request time and never stored or redistributed. The NLOD attribution is registered in the Data attribution popover. -- **Kartverket ground heights for Statens vegvesen cameras.** The camera feed's points are 2D, so the loader asks Kartverket's keyless point-height API (`ws.geonorge.no/hoydedata/v1/punkt`, CC BY 4.0) for the terrain height at each camera, 50 points per request with up to six in flight (about 17 requests, ~5 s, for the whole country). Points a full batch answers `null` for are retried once in batches of 10, since the API sometimes drops points from large batches; points still without data keep the flat 150 m prior. Over water the API returns the sea-floor depth (`dybdekurver` / `Havflate`), which is replaced by 0 m, the water surface. Heights are cached in memory by position, so the 15-minute catalog refresh only asks about new or moved cameras. A catalog load waits at most 12 s; a slower lookup keeps filling the cache for the next refresh. Only coordinates are sent. `CCTV_VEGVESEN_HEIGHTS=0` turns the lookup off. The Kartverket credit is shown beside the Statens vegvesen entry in the Data attribution popover. +- **Kartverket ground heights for Statens vegvesen cameras.** The camera feed's points are 2D, so the loader asks Kartverket's keyless point-height API (`ws.geonorge.no/hoydedata/v1/punkt`, CC BY 4.0) for the terrain height at each camera. Requests carry 20 points: the API accepts 50, but above ~20 it answers the extra points from contour lines (`hoydekurver`, metres to ~20 m off) or not at all. Eight requests run at a time, about 36 for the whole country (~6 s). Answers are matched by the coordinates the API echoes and recorded only for points that request asked about. Over water the API returns the sea-floor depth (`dybdekurver` or `Havflate`), which is replaced by 0 m, the water surface; for the ~14 bridge cameras that is below the deck, but much closer than the sea floor or the flat prior. Heights are cached in memory by position for the life of the server, and points with no height (a few ferry quays and bridges) for 6 h, so the 15-minute catalog refresh only asks about new or moved cameras. The first 429, 5xx or network failure stops the lookup, with no retries, and no new lookup starts until the backoff ends (Retry-After when sent, else 10 min; bounded to 1 min–1 h). The lane waits at most 12 s, counted from the start of the loader; a slower lookup keeps filling the cache for the next refresh. Only coordinates are sent, with the same User-Agent as the other CCTV requests. `CCTV_VEGVESEN_HEIGHTS=0` turns the lookup off. The Kartverket credit is shown beside the Statens vegvesen entry in the Data attribution popover. - **Camera frame content.** Public camera frames can contain people, vehicles and license plates, and the camera layer does nothing with them beyond putting them on the map. A successful upstream response is relayed as the provider served it: nothing in the camera pipeline blurs, redacts, enhances or restores it, and nothing there runs face, plate or object detection or otherwise recognises or classifies what is in the picture. (The ALPR layer described below maps where plate-reader cameras are, from locations contributors tagged in OpenStreetMap; it reads nothing from any camera frame.) When an upstream has no frame for a camera, what you see instead is a Google Street View still or the app's own placeholder card; the panel's source badge names which of them you are looking at, and a refresh that fails after a frame has arrived leaves the last good picture on screen rather than replacing it. The browser resamples each frame for display: onto a fixed 1920x1080 surface for the camera's projection plane in the scene, and onto a 192x108 canvas for the small ambient card. That scaling is the only change made to the picture, and the only pixels the camera layer reads back are a 64x36 downscale hashed to tell whether the picture has changed. Live streams are piped through; the still path reads the image into memory to pass it on — TxDOT wraps its JPEG in a JSON envelope, which is decoded there — and no frame is written to disk (the disk cache, `.gev-cache/`, holds feed JSON for other layers and never camera imagery). Outside that pipeline, the voice assistant is the one feature that sends imagery anywhere: when it answers a question about what is in view it may attach a downscaled screenshot of the rendered scene — including any camera plane on screen — to the request it sends to the voice model. Frames are fetched only from server-registered camera URLs — a client cannot hand the proxy a URL (see [SECURITY.md](SECURITY.md)). What the app does stays inside the line README draws: no named-person search, no face recognition, no tracking individuals. - **Transit (GTFS-Realtime).** The proxy requests only registered operator feeds, validates redirects, bounds responses, and uses conditional requests, short snapshot caching and failure backoff. All seven registered regions are live-enabled; each can be disabled independently. Vehicles use delayed playback between reports, with report age and waiting states shown separately. Session-local selected trails are available for every feed. MBTA additionally retains up to 15 minutes of recently observed positions in the running proxy, so available history can survive a browser reload; server restart clears it. History reads never fetch upstream, and other feeds have no proxy retention. Nothing is archived to disk or collected in the background. Operator text credits are included as a courtesy and do not imply endorsement. Heights are aligned to work with Google 3D tiles; subway markers are shown at street level because these feeds do not supply depth. - **Radio Browser.** `/api/radio/stations` discovers official API mirrors, makes bounded and coalesced healthy/geolocated HTTPS-station queries, caches the normalized public-domain directory for 45 minutes, and may serve the last good catalog for up to seven days during an outage. Refreshes must meet minimum accepted-query and station coverage before replacing a warm catalog; schema-valid responses whose rows all fail the product's health policy do not count as successful queries. A usable partial cold catalog is explicitly `DEGRADED`, and malformed or empty successful payloads are rejected atomically. Every directory and click-count request rejects redirects, validates all resolved addresses as globally routable (including reserved/documentation IPv4 and special/non-global IPv6 exclusions), and pins the TLS connection to a validated address. Only MP3/AAC non-HLS directory rows with public HTTPS stream targets are returned; favicons are intentionally omitted. Pressing play connects one browser audio element directly to the selected broadcaster and calls the directory's click counter through known-ID-only `POST /api/radio/click/:uuid`. GEV never proxies, caches, records, bundles, or redistributes audio. Radio Browser supplies station-level tags, not dependable current-song or upcoming-program metadata, so Radio filtering never claims either. Direct playback exposes the listener's IP address to the broadcaster, whose own stream terms apply. diff --git a/server/providers/cctv/constants.js b/server/providers/cctv/constants.js index 7609a3fc5..6e9c6cd04 100644 --- a/server/providers/cctv/constants.js +++ b/server/providers/cctv/constants.js @@ -268,28 +268,37 @@ export const NORWAY_ANCHORS = [ ]; /** Kartverket's keyless point-height API (CC BY 4.0): terrain height above - * sea level (NN2000) for up to 50 points per request, used as the Vegvesen - * cameras' ground prior because the camera feed's points are 2D. */ + * sea level (NN2000), used as the Vegvesen cameras' ground prior because the + * camera feed's points are 2D. */ export const KARTVERKET_HEIGHT_URL = 'https://ws.geonorge.no/hoydedata/v1/punkt'; -export const KARTVERKET_MAX_POINTS = 50; -/** Batch size for the one retry of points a full batch answered null for. */ -export const KARTVERKET_RETRY_POINTS = 10; -/** Batches in flight at once. The whole country is ~17 batches of ~2 s each - * (spread-out points are slow to resolve), so six keep a cold lookup ~6 s. */ -export const KARTVERKET_CONCURRENCY = 6; -/** Deadline for one complete lookup, shared by every batch. */ +/** Points per request. The API takes up to 50, but above ~20 it answers the + * extra points from contour lines (`hoydekurver`, metres to ~20 m off) or not + * at all; at 20 nearly every camera resolves from the 1 m terrain model. */ +export const KARTVERKET_MAX_POINTS = 20; +/** Requests in flight at once. The whole country is ~36 requests of ~1-2 s, + * so a cold lookup takes ~6 s. */ +export const KARTVERKET_CONCURRENCY = 8; +/** Deadline for one complete lookup, shared by every request. */ export const KARTVERKET_LOOKUP_TIMEOUT_MS = 30 * 1000; -/** How long a catalog load waits for heights. Kept under the per-provider - * fetch timeout so this pack is never the catalog's slowest lane; a lookup - * still running after this keeps filling the cache, and the next refresh - * applies the rest. At app start, with every pack loading at once, a cold - * lookup takes ~10 s. */ +/** The Vegvesen lane's budget, measured from the start of the loader (catalog + * fetch included), for waiting on heights. Under the 15 s per-provider fetch + * timeout so this pack is never the catalog's slowest lane. A lookup still + * running after it keeps filling the cache for the next refresh. */ export const KARTVERKET_CATALOG_WAIT_MS = 12 * 1000; +/** A point Kartverket has no height for (some ferry quays and bridges) is not + * asked about again for this long. */ +export const KARTVERKET_NULL_TTL_MS = 6 * 60 * 60 * 1000; +/** After a 429, a 5xx or a network failure, no lookup starts for this long, + * or for the Retry-After the service sent, bounded to [1 min, 1 h]. */ +export const KARTVERKET_FAILURE_BACKOFF_MS = 10 * 60 * 1000; +export const KARTVERKET_MAX_BACKOFF_MS = 60 * 60 * 1000; export const KARTVERKET_MAX_RESPONSE_BYTES = 256 * 1024; /** Plausible Norwegian terrain: below sea level only at the shore, and no road - * camera sits above ~1,500 m. Anything outside is treated as no answer. */ + * camera sits above ~1,500 m. Anything outside is treated as no height. */ export const KARTVERKET_HEIGHT_RANGE_M = Object.freeze({ min: -20, max: 2500 }); +/** Same identity the other CCTV upstream requests send. */ +export const KARTVERKET_USER_AGENT = 'gods-eye-view-cctv-proxy/1.0'; /** DelDOT CCTV: one keyless statewide JSON catalog; live video via RTMP-over-HTTP (rtmpt:80). */ export const DELDOT_CCTV_URL = 'https://tmc.deldot.gov/json/videocamera.json'; diff --git a/server/providers/cctv/sources.js b/server/providers/cctv/sources.js index 281ddde85..15f6df74b 100644 --- a/server/providers/cctv/sources.js +++ b/server/providers/cctv/sources.js @@ -67,10 +67,13 @@ import { NORWAY_ANCHORS, KARTVERKET_HEIGHT_URL, KARTVERKET_MAX_POINTS, - KARTVERKET_RETRY_POINTS, KARTVERKET_CONCURRENCY, KARTVERKET_LOOKUP_TIMEOUT_MS, KARTVERKET_CATALOG_WAIT_MS, + KARTVERKET_NULL_TTL_MS, + KARTVERKET_FAILURE_BACKOFF_MS, + KARTVERKET_MAX_BACKOFF_MS, + KARTVERKET_USER_AGENT, KARTVERKET_MAX_RESPONSE_BYTES, KARTVERKET_HEIGHT_RANGE_M, } from './constants.js'; @@ -1811,6 +1814,7 @@ export function vegvesenCameraToSource(feature) { * @returns {Promise>} Normalized camera source objects. */ export async function loadVegvesenSourcesFromOpenData() { + const startedAt = Date.now(); try { const endpoint = process.env.CCTV_VEGVESEN_URL || DEFAULT_VEGVESEN_CCTV_URL; const resp = await fetch(endpoint, { @@ -1856,7 +1860,12 @@ export async function loadVegvesenSourcesFromOpenData() { ? Math.max(8, Math.min(1000, Math.floor(maxRaw))) : DEFAULT_VEGVESEN_MAX_SOURCES; const prioritized = prioritizeSources(cameras, maxCount, NORWAY_ANCHORS); - await applyVegvesenGroundHeights(prioritized); + await applyVegvesenGroundHeights(prioritized, { + waitMs: Math.max( + 0, + KARTVERKET_CATALOG_WAIT_MS - (Date.now() - startedAt), + ), + }); console.log( `[CCTV] Loaded Vegvesen camera sources: ${cameras.length} (using nearest ${prioritized.length})`, ); @@ -1871,49 +1880,79 @@ export async function loadVegvesenSourcesFromOpenData() { } /** - * Terrain heights already resolved, keyed by rounded position. Camera - * positions rarely change, so after the first catalog load a 15-minute refresh - * asks Kartverket only for new or moved cameras. Bounded well above the ~900 - * cameras the pack can load. + * Kartverket answers by rounded position: `{z, expires}`, where z is metres or + * null for "no height there". Heights never expire (camera positions barely + * change); nulls expire after KARTVERKET_NULL_TTL_MS. The 15-minute catalog + * refresh therefore only asks about new or moved cameras. Bounded well above + * the ~900 cameras the pack can load. */ const kartverketHeightCache = new Map(); const KARTVERKET_CACHE_LIMIT = 5000; /** The lookup in flight, so overlapping catalog loads never start a second. */ let kartverketPending = null; +/** No lookup starts before this time (epoch ms) after a transport failure. */ +let kartverketBackoffUntil = 0; /** Cache key for one position, at the ~0.1 m precision the request sends. */ const kartverketKey = (lon, lat) => `${lon.toFixed(6)},${lat.toFixed(6)}`; -/** Forget cached heights and any lookup in flight. Tests only. */ +/** Forget cached heights, the backoff and any lookup in flight. Tests only. */ export function clearKartverketHeightCache() { kartverketHeightCache.clear(); kartverketPending = null; + kartverketBackoffUntil = 0; +} + +/** + * How long to back off after a failed request: the Retry-After the service + * sent (seconds or an HTTP date), else the default, bounded to [1 min, 1 h]. + * + * @param {?string} retryAfter - The Retry-After header, if any. + * @param {number} now - Epoch ms. + * @returns {number} Milliseconds. + */ +export function kartverketBackoffMs(retryAfter, now = Date.now()) { + let ms = KARTVERKET_FAILURE_BACKOFF_MS; + const text = String(retryAfter ?? '').trim(); + if (/^\d+$/.test(text)) ms = Number(text) * 1000; + else if (text && Number.isFinite(Date.parse(text))) { + ms = Date.parse(text) - now; + } + return Math.max(60 * 1000, Math.min(KARTVERKET_MAX_BACKOFF_MS, ms)); } /** * Terrain heights above sea level (NN2000) from Kartverket's keyless - * point-height API, in batches of up to 50 points. Answers are matched back by - * the coordinates the API echoes, not by position in the list. Each answer is - * written to `into` as its batch lands, so a caller that stops waiting still - * keeps what has arrived. One deadline covers every request. + * point-height API, KARTVERKET_MAX_POINTS points per request. Each answer is + * matched back by the coordinates the API echoes, and only for points that + * request asked about, then reported through `into.set(key, z)` as its + * request lands, so a caller that stops waiting keeps what has arrived. `z` is + * metres, or null when the API has no usable height there. * - * Two quirks of the live API are handled here: - * - A large batch sometimes answers `z: null` for a point it resolves when - * asked again (Valdresflye: null in a 50-point batch, 1,390.7 m alone), so - * every point still missing after the first pass is retried once in batches - * of KARTVERKET_RETRY_POINTS. A point still null then has no data. - * - Over water `/punkt` returns the depth of the sea floor (`dybdekurver`, - * terrain `Havflate`; −547 m under Nordhordlandsbrua). A camera on a bridge - * or quay stands over the water surface, so those answers count as 0 m. + * Over water `/punkt` returns the depth of the sea floor (`datakilde: + * dybdekurver`, `terreng: Havflate`; −547 m under Nordhordlandsbrua). Either + * marker counts as 0 m, the water surface. For the ~14 bridge cameras that is + * below the deck, but far closer than the sea floor or a flat prior. + * + * The first transport failure (a non-200 answer, a network error or the + * shared deadline) stops the lookup: no further requests start, and the + * failure is returned so the caller can back off. Points in failed requests + * are not reported, so they are asked again after the backoff. * * @param {Array<{lon:number, lat:number}>} points - * @param {{into?: Map, timeoutMs?: number}} [options] - * @returns {Promise>} kartverketKey(lon, lat) → metres. + * @param {{into?: {set: Function}, timeoutMs?: number}} [options] + * @returns {Promise<{heights: Map, failure: ?{status: ?number, retryAfter: ?string}}>} + * `heights` is `into` when it is a Map, else a Map of what was reported. */ export async function fetchKartverketHeights( points, { into = new Map(), timeoutMs = KARTVERKET_LOOKUP_TIMEOUT_MS } = {}, ) { + const heights = into instanceof Map ? into : new Map(); + const report = (key, z) => { + if (heights !== into) heights.set(key, z); + into.set(key, z); + }; const unique = Array.from( new Map( (points || []) @@ -1921,9 +1960,15 @@ export async function fetchKartverketHeights( .map((p) => [kartverketKey(p.lon, p.lat), p]), ).values(), ); - if (!unique.length) return into; + let failure = null; + if (!unique.length) return { heights, failure }; + const batches = []; + for (let i = 0; i < unique.length; i += KARTVERKET_MAX_POINTS) { + batches.push(unique.slice(i, i + KARTVERKET_MAX_POINTS)); + } const signal = AbortSignal.timeout(timeoutMs); const fetchBatch = async (batch) => { + const asked = new Set(batch.map((p) => kartverketKey(p.lon, p.lat))); const url = new URL(KARTVERKET_HEIGHT_URL); url.searchParams.set( 'punkter', @@ -1933,7 +1978,10 @@ export async function fetchKartverketHeights( ); url.searchParams.set('koordsys', '4258'); const resp = await fetch(url, { - headers: { Accept: 'application/json' }, + headers: { + Accept: 'application/json', + 'User-Agent': KARTVERKET_USER_AGENT, + }, redirect: 'error', signal, }); @@ -1943,7 +1991,10 @@ export async function fetchKartverketHeights( } catch { /* no-op */ } - throw new Error(`HTTP ${resp.status}`); + const error = new Error(`HTTP ${resp.status}`); + error.status = resp.status; + error.retryAfter = resp.headers.get('retry-after'); + throw error; } const payload = await readResponseJsonCapped( resp, @@ -1952,102 +2003,120 @@ export async function fetchKartverketHeights( for (const point of Array.isArray(payload?.punkter) ? payload.punkter : []) { - // `z: null` means no answer for that point; Number(null) would read 0. - if (point?.z === null || point?.z === undefined || point?.z === '') - continue; const x = toFiniteNumber(point?.x); const y = toFiniteNumber(point?.y); - let z = toFiniteNumber(point?.z); - if (![x, y, z].every(Number.isFinite)) continue; - if (point?.datakilde === 'dybdekurver' || point?.terreng === 'Havflate') - z = 0; + if (!Number.isFinite(x) || !Number.isFinite(y)) continue; + const key = kartverketKey(x, y); + // Only the points this request asked about: an odd answer can neither + // grow the cache nor overwrite another camera. + if (!asked.has(key)) continue; + asked.delete(key); + // `z: null` means no height there; Number(null) would read 0. + const raw = point?.z; + let z = + raw === null || raw === undefined || raw === '' + ? NaN + : toFiniteNumber(raw); if ( - z < KARTVERKET_HEIGHT_RANGE_M.min || - z > KARTVERKET_HEIGHT_RANGE_M.max - ) - continue; - into.set(kartverketKey(x, y), z); + Number.isFinite(z) && + (point?.datakilde === 'dybdekurver' || point?.terreng === 'Havflate') + ) { + z = 0; + } + const usable = + Number.isFinite(z) && + z >= KARTVERKET_HEIGHT_RANGE_M.min && + z <= KARTVERKET_HEIGHT_RANGE_M.max; + report(key, usable ? z : null); } }; - let failed = 0; - const runPass = async (list, size) => { - const batches = []; - for (let i = 0; i < list.length; i += size) { - batches.push(list.slice(i, i + size)); - } - let next = 0; - const worker = async () => { - while (next < batches.length && !signal.aborted) { - const batch = batches[next++]; - try { - await fetchBatch(batch); - } catch (error) { - failed += 1; - if (failed === 1) { - console.warn( - '[CCTV] Kartverket height lookup failed:', - error?.message || error, - ); - } + let next = 0; + const worker = async () => { + while (next < batches.length && !failure) { + const batch = batches[next++]; + try { + await fetchBatch(batch); + } catch (error) { + if (!failure) { + failure = { + status: Number.isFinite(error?.status) ? error.status : null, + retryAfter: error?.retryAfter ?? null, + }; + console.warn( + '[CCTV] Kartverket height lookup failed:', + error?.message || error, + ); } } - }; - await Promise.all( - Array.from( - { length: Math.min(KARTVERKET_CONCURRENCY, batches.length) }, - worker, - ), - ); + } }; - await runPass(unique, KARTVERKET_MAX_POINTS); - const missing = unique.filter((p) => !into.has(kartverketKey(p.lon, p.lat))); - if (missing.length && !signal.aborted) { - await runPass(missing, KARTVERKET_RETRY_POINTS); - } - return into; + await Promise.all( + Array.from( + { length: Math.min(KARTVERKET_CONCURRENCY, batches.length) }, + worker, + ), + ); + return { heights, failure }; } /** * Give each Vegvesen camera its own terrain height as the ground prior, in - * place. Waits at most `waitMs` for heights it does not have cached; a lookup + * place. Waits at most `waitMs` for positions it has no answer for; a lookup * still running then keeps filling the cache for the next catalog refresh. - * Cameras without a height keep the flat prior. Never throws: a failed lookup - * only means the old behavior. CCTV_VEGVESEN_HEIGHTS=0 turns it off. + * Cameras without a height keep the flat prior. After a transport failure no + * lookup starts until the backoff ends, so a failing Kartverket costs one + * attempt, not one per refresh. Never throws. CCTV_VEGVESEN_HEIGHTS=0 turns it + * off. * * @param {Array} cameras - Sources from vegvesenCameraToSource(). - * @param {{waitMs?: number}} [options] + * @param {{waitMs?: number, now?: () => number}} [options] * @returns {Promise} How many cameras got a terrain height. */ export async function applyVegvesenGroundHeights( cameras, - { waitMs = KARTVERKET_CATALOG_WAIT_MS } = {}, + { waitMs = KARTVERKET_CATALOG_WAIT_MS, now = Date.now } = {}, ) { if (String(process.env.CCTV_VEGVESEN_HEIGHTS || '1').trim() === '0') return 0; const list = Array.isArray(cameras) ? cameras : []; - const missing = list.filter( - (c) => !kartverketHeightCache.has(kartverketKey(c.lon, c.lat)), - ); - if (missing.length) { - if (!kartverketPending) { - if ( - kartverketHeightCache.size + missing.length > - KARTVERKET_CACHE_LIMIT - ) { - kartverketHeightCache.clear(); - } - kartverketPending = fetchKartverketHeights(missing, { - into: kartverketHeightCache, - }) - .catch((error) => { - console.warn( - '[CCTV] Kartverket height lookup error:', - error?.message || error, - ); - }) - .finally(() => { - kartverketPending = null; - }); + const startedAt = now(); + const missing = list.filter((c) => { + const entry = kartverketHeightCache.get(kartverketKey(c.lon, c.lat)); + return !entry || entry.expires <= startedAt; + }); + if ( + missing.length && + !kartverketPending && + startedAt >= kartverketBackoffUntil + ) { + if (kartverketHeightCache.size + missing.length > KARTVERKET_CACHE_LIMIT) { + kartverketHeightCache.clear(); } + const sink = { + set: (key, z) => + kartverketHeightCache.set(key, { + z, + expires: z === null ? now() + KARTVERKET_NULL_TTL_MS : Infinity, + }), + }; + kartverketPending = fetchKartverketHeights(missing, { into: sink }) + .then(({ failure }) => { + if (failure) { + kartverketBackoffUntil = + now() + kartverketBackoffMs(failure.retryAfter, now()); + } + }) + .catch((error) => { + kartverketBackoffUntil = now() + KARTVERKET_FAILURE_BACKOFF_MS; + console.warn( + '[CCTV] Kartverket height lookup error:', + error?.message || error, + ); + }) + .finally(() => { + kartverketPending = null; + }); + } + if (missing.length && kartverketPending && waitMs > 0) { let timer; await Promise.race([ kartverketPending, @@ -2060,7 +2129,9 @@ export async function applyVegvesenGroundHeights( } let applied = 0; for (const camera of list) { - const z = kartverketHeightCache.get(kartverketKey(camera.lon, camera.lat)); + const z = kartverketHeightCache.get( + kartverketKey(camera.lon, camera.lat), + )?.z; if (!Number.isFinite(z)) continue; camera.groundElevationM = Math.round(z * 10) / 10; applied += 1; diff --git a/src/data/cctvVegvesen.test.mjs b/src/data/cctvVegvesen.test.mjs index ffb25aec3..47ae732d1 100644 --- a/src/data/cctvVegvesen.test.mjs +++ b/src/data/cctvVegvesen.test.mjs @@ -5,14 +5,18 @@ import { applyVegvesenGroundHeights, clearKartverketHeightCache, fetchKartverketHeights, + kartverketBackoffMs, loadVegvesenSourcesFromOpenData, vegvesenCameraToSource, } from '../../server/providers/cctv/sources.js'; import { DEFAULT_VEGVESEN_CCTV_URL, KARTVERKET_HEIGHT_URL, + KARTVERKET_CONCURRENCY, + KARTVERKET_FAILURE_BACKOFF_MS, KARTVERKET_MAX_POINTS, - KARTVERKET_RETRY_POINTS, + KARTVERKET_NULL_TTL_MS, + KARTVERKET_USER_AGENT, VEGVESEN_IMAGE_ORIGIN, } from '../../server/providers/cctv/constants.js'; import { createCctvCatalog } from '../../server/providers/cctv/catalog.js'; @@ -244,145 +248,261 @@ const withHeightsEnv = (t) => { t.mock.method(console, 'warn', () => {}); }; -test('Kartverket heights come in batches of 50, matched by echoed coordinates', async (t) => { +const sentPoints = (url) => + JSON.parse(new URL(url).searchParams.get('punkter')); + +test('Kartverket heights come in batches of 20, matched by echoed coordinates', async (t) => { withHeightsEnv(t); const requested = []; - t.mock.method(globalThis, 'fetch', async (url) => { + t.mock.method(globalThis, 'fetch', async (url, init) => { requested.push(String(url)); assert.ok(String(url).startsWith(KARTVERKET_HEIGHT_URL)); + assert.equal(init.headers['User-Agent'], KARTVERKET_USER_AGENT); // Reply in reverse order: matching must not depend on list position. const response = kartverketResponse(url, (x) => - x > 7.05 ? null : Math.round((x - 7) * 10000), + x > 7.04 ? null : Math.round((x - 7) * 10000), ); const body = await response.json(); body.punkter.reverse(); return Response.json(body); }); - const points = Array.from({ length: 120 }, (_, i) => cameraAt(i)); - const heights = await fetchKartverketHeights(points); - const sizes = requested.map( - (url) => JSON.parse(new URL(url).searchParams.get('punkter')).length, + const { heights, failure } = await fetchKartverketHeights( + Array.from({ length: 50 }, (_, i) => cameraAt(i)), ); - // Three full batches, then one retry pass over the 69 points with no height. - assert.deepEqual(sizes.slice(0, 3), [KARTVERKET_MAX_POINTS, 50, 20]); - assert.equal( - sizes.slice(3).reduce((a, b) => a + b, 0), - 69, + assert.equal(failure, null); + // One pass, no retries: 20 + 20 + 10. + assert.deepEqual( + requested.map((url) => sentPoints(url).length), + [KARTVERKET_MAX_POINTS, 20, 10], ); - assert.ok(sizes.slice(3).every((n) => n <= KARTVERKET_RETRY_POINTS)); for (const url of requested) { assert.equal(new URL(url).searchParams.get('koordsys'), '4258'); } - // Points past x = 7.05 have no height (outside coverage) and are left out. - assert.equal(heights.size, 51); + assert.equal(heights.size, 50); assert.equal(heights.get('7.010000,60.010000'), 100); + // Past x = 7.04 the API has no height: recorded as null, never as 0. + assert.equal(heights.get('7.049000,60.049000'), null); }); -test('points a full batch answers null for are retried once in small batches', async (t) => { - withHeightsEnv(t); - const sizes = []; - t.mock.method(globalThis, 'fetch', async (url) => { - const points = JSON.parse(new URL(url).searchParams.get('punkter')); - sizes.push(points.length); - // A full batch drops the first two points; a small batch answers them, - // except the one point that has no data at all. - const full = points.length > KARTVERKET_RETRY_POINTS; - return kartverketResponse(url, (x) => { - if (Math.abs(x - 7.002) < 1e-9) return null; - if (full && x < 7.002) return null; - return 500; - }); - }); - const heights = await fetchKartverketHeights( - Array.from({ length: 60 }, (_, i) => cameraAt(i)), - ); - assert.deepEqual(sizes, [50, 10, 3]); - assert.equal(heights.size, 59); - assert.equal(heights.get('7.000000,60.000000'), 500); - assert.equal(heights.has('7.002000,60.002000'), false); -}); - -test('a sea-floor depth counts as the water surface', async (t) => { +test('only the points a request asked about are recorded', async (t) => { withHeightsEnv(t); t.mock.method(globalThis, 'fetch', async (url) => { - const points = JSON.parse(new URL(url).searchParams.get('punkter')); + const points = sentPoints(url); return Response.json({ koordsys: 4258, - punkter: points.map(([x, y], i) => - i === 0 - ? { x, y, z: -547.1, datakilde: 'dybdekurver', terreng: 'Havflate' } - : { x, y, z: -0.2, datakilde: 'dtm1', terreng: 'ÅpentOmråde' }, - ), + punkter: [ + ...points.map(([x, y]) => ({ x, y, z: 10 })), + { x: 7.5, y: 60.5, z: 999 }, // never asked + ], }); }); - const heights = await fetchKartverketHeights([cameraAt(0), cameraAt(1)]); - // Under a bridge: the surface, not the sea floor. A quay just below the - // datum on land data stays as measured. - assert.equal(heights.get('7.000000,60.000000'), 0); - assert.equal(heights.get('7.001000,60.001000'), -0.2); + const { heights } = await fetchKartverketHeights([cameraAt(0)]); + assert.deepEqual([...heights.entries()], [['7.000000,60.000000', 10]]); }); -test('implausible heights and points outside Norway are ignored', async (t) => { +test('either water marker counts as the surface', async (t) => { + withHeightsEnv(t); + t.mock.method(globalThis, 'fetch', async (url) => { + const [a, b, c] = sentPoints(url); + return Response.json({ + koordsys: 4258, + punkter: [ + { + x: a[0], + y: a[1], + z: -547.1, + datakilde: 'dybdekurver', + terreng: null, + }, + { x: b[0], y: b[1], z: -9.2, datakilde: 'dtm1', terreng: 'Havflate' }, + { + x: c[0], + y: c[1], + z: -0.2, + datakilde: 'dtm1', + terreng: 'ÅpentOmråde', + }, + ], + }); + }); + const { heights } = await fetchKartverketHeights([ + cameraAt(0), + cameraAt(1), + cameraAt(2), + ]); + assert.equal(heights.get('7.000000,60.000000'), 0); + assert.equal(heights.get('7.001000,60.001000'), 0); + // A quay just below the datum on land data stays as measured. + assert.equal(heights.get('7.002000,60.002000'), -0.2); +}); + +test('heights outside -20..2500 m and points outside Norway are not used', async (t) => { withHeightsEnv(t); const requested = []; t.mock.method(globalThis, 'fetch', async (url) => { requested.push(String(url)); - return kartverketResponse(url, () => 99999); + const zs = [-20, -20.5, 2500, 2500.5]; + const points = sentPoints(url); + return Response.json({ + koordsys: 4258, + punkter: points.map(([x, y], i) => ({ x, y, z: zs[i] })), + }); }); - const heights = await fetchKartverketHeights([ + const { heights } = await fetchKartverketHeights([ + cameraAt(0), cameraAt(1), + cameraAt(2), + cameraAt(3), { lat: 55.68, lon: 12.57 }, // Copenhagen: never sent ]); - assert.equal(heights.size, 0); - // The first pass and its one retry, each carrying only the Norway point. - assert.equal(requested.length, 2); - for (const url of requested) { - assert.equal( - JSON.parse(new URL(url).searchParams.get('punkter')).length, - 1, - ); - } + assert.equal(requested.length, 1); + assert.equal(sentPoints(requested[0]).length, 4); + assert.deepEqual( + [0, 1, 2, 3].map((i) => + heights.get( + `${(7 + i * 0.001).toFixed(6)},${(60 + i * 0.001).toFixed(6)}`, + ), + ), + [-20, null, 2500, null], + ); }); -test('cameras get their terrain height, and a refresh reuses the cache', async (t) => { +test('no more than eight requests are in flight', async (t) => { + withHeightsEnv(t); + let inFlight = 0; + let peak = 0; + t.mock.method(globalThis, 'fetch', async (url) => { + inFlight += 1; + peak = Math.max(peak, inFlight); + await new Promise((resolve) => setTimeout(resolve, 5)); + inFlight -= 1; + return kartverketResponse(url, () => 1); + }); + const { heights } = await fetchKartverketHeights( + Array.from({ length: 400 }, (_, i) => cameraAt(i)), + ); + assert.equal(heights.size, 400); + assert.equal(peak, KARTVERKET_CONCURRENCY); +}); + +test('a transport failure stops the lookup: no retries, no new requests', async (t) => { withHeightsEnv(t); let calls = 0; - t.mock.method(globalThis, 'fetch', async (url) => { + t.mock.method(globalThis, 'fetch', async () => { calls += 1; + return new Response('busy', { + status: 429, + headers: { 'Retry-After': '120' }, + }); + }); + const { heights, failure } = await fetchKartverketHeights( + Array.from({ length: 400 }, (_, i) => cameraAt(i)), + ); + // The first wave is already in flight when the first failure lands; nothing + // after it starts, and no point is resent. + assert.equal(calls, KARTVERKET_CONCURRENCY); + assert.equal(heights.size, 0); + assert.deepEqual(failure, { status: 429, retryAfter: '120' }); +}); + +test('the lookup deadline bounds a hanging service', async (t) => { + withHeightsEnv(t); + t.mock.method( + globalThis, + 'fetch', + (url, init) => + new Promise((_, reject) => { + init.signal.addEventListener('abort', () => reject(init.signal.reason)); + }), + ); + const started = Date.now(); + const { heights, failure } = await fetchKartverketHeights([cameraAt(0)], { + timeoutMs: 30, + }); + assert.ok(Date.now() - started < 2000); + assert.equal(heights.size, 0); + assert.equal(failure.status, null); +}); + +test('Retry-After sets the backoff, bounded to [1 min, 1 h]', () => { + const now = Date.parse('2026-10-06T12:00:00Z'); + assert.equal(kartverketBackoffMs('120', now), 120_000); + assert.equal(kartverketBackoffMs('5', now), 60_000); + assert.equal(kartverketBackoffMs('86400', now), 3_600_000); + assert.equal( + kartverketBackoffMs('Mon, 06 Oct 2026 12:05:00 GMT', now), + 300_000, + ); + assert.equal(kartverketBackoffMs(null, now), KARTVERKET_FAILURE_BACKOFF_MS); + assert.equal(kartverketBackoffMs('soon', now), KARTVERKET_FAILURE_BACKOFF_MS); +}); + +test('heights are cached, and misses are not asked again until they expire', async (t) => { + withHeightsEnv(t); + let clock = 1_000_000; + const now = () => clock; + const asked = []; + t.mock.method(globalThis, 'fetch', async (url) => { + asked.push(sentPoints(url).length); return kartverketResponse(url, (x) => (x < 7.002 ? 1043.26 : null)); }); const cameras = [cameraAt(0), cameraAt(1), cameraAt(5)]; - assert.equal(await applyVegvesenGroundHeights(cameras), 2); + assert.equal(await applyVegvesenGroundHeights(cameras, { now }), 2); assert.deepEqual( cameras.map((c) => c.groundElevationM), [1043.3, 1043.3, 150], ); - // One full pass plus one retry for the point with no height. - assert.equal(calls, 2); - // Same positions on the next catalog refresh: answered from the cache. - const again = [cameraAt(0), cameraAt(1)]; - assert.equal(await applyVegvesenGroundHeights(again), 2); - assert.equal(calls, 2); + // The next refresh asks nothing: heights and the miss are both cached. + assert.equal( + await applyVegvesenGroundHeights([cameraAt(0), cameraAt(1), cameraAt(5)], { + now, + }), + 2, + ); + assert.deepEqual(asked, [3]); + // Once the miss expires, only that point is asked again. + clock += KARTVERKET_NULL_TTL_MS + 1; + await applyVegvesenGroundHeights([cameraAt(0), cameraAt(5)], { now }); + assert.deepEqual(asked, [3, 1]); }); -test('a failed or disabled lookup keeps the flat prior', async (t) => { +test('after a failure no lookup starts until the backoff ends', async (t) => { withHeightsEnv(t); + let clock = 1_000_000; + const now = () => clock; const fetchMock = t.mock.method(globalThis, 'fetch', async () => { throw new Error('offline'); }); const cameras = [cameraAt(0)]; - assert.equal(await applyVegvesenGroundHeights(cameras), 0); + assert.equal(await applyVegvesenGroundHeights(cameras, { now }), 0); assert.equal(cameras[0].groundElevationM, 150); + assert.equal(fetchMock.mock.callCount(), 1); - fetchMock.mock.mockImplementation( - async () => new Response('nope', { status: 503 }), + // Refreshes inside the backoff cost nothing. + clock += KARTVERKET_FAILURE_BACKOFF_MS - 1; + assert.equal(await applyVegvesenGroundHeights([cameraAt(0)], { now }), 0); + assert.equal(fetchMock.mock.callCount(), 1); + + // After it, the failed point is asked again. + clock += 2; + fetchMock.mock.mockImplementation(async (url) => + kartverketResponse(url, () => 321), ); + const later = [cameraAt(0)]; + assert.equal(await applyVegvesenGroundHeights(later, { now }), 1); + assert.equal(later[0].groundElevationM, 321); + assert.equal(fetchMock.mock.callCount(), 2); +}); + +test('CCTV_VEGVESEN_HEIGHTS=0 skips the lookup', async (t) => { + withHeightsEnv(t); + const fetchMock = t.mock.method(globalThis, 'fetch', async (url) => + kartverketResponse(url, () => 5), + ); + process.env.CCTV_VEGVESEN_HEIGHTS = '0'; + const cameras = [cameraAt(0)]; assert.equal(await applyVegvesenGroundHeights(cameras), 0); assert.equal(cameras[0].groundElevationM, 150); - - process.env.CCTV_VEGVESEN_HEIGHTS = '0'; - fetchMock.mock.resetCalls(); - assert.equal(await applyVegvesenGroundHeights(cameras), 0); assert.equal(fetchMock.mock.callCount(), 0); }); diff --git a/src/data/dataCredits.js b/src/data/dataCredits.js index 8ad585eca..aa8e8f489 100644 --- a/src/data/dataCredits.js +++ b/src/data/dataCredits.js @@ -246,7 +246,8 @@ export const DATA_CREDITS = [ 'Road cameras (Norway): contains data under the ' + 'Norwegian licence for Open Government data (NLOD) ' + 'distributed by Statens vegvesen · camera ground heights © ' + - 'Kartverket (CC BY 4.0)', + 'Kartverket ' + + '(CC BY 4.0)', }, { key: 'gbfs', From 15208b5c40323e62523a411b0035f1cf4e4e3d2a Mon Sep 17 00:00:00 2001 From: Trygve Hemminghytt <274629477+trygvehhh@users.noreply.github.com> Date: Sun, 18 Oct 2026 20:03:24 +0200 Subject: [PATCH 3/9] docs: move the changelog entry to changelog.d --- CHANGELOG.md | 14 -------------- .../945-cctv-vegvesen-kartverket-heights.md | 14 ++++++++++++++ 2 files changed, 14 insertions(+), 14 deletions(-) create mode 100644 changelog.d/945-cctv-vegvesen-kartverket-heights.md diff --git a/CHANGELOG.md b/CHANGELOG.md index 58e2f1798..98d32365f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -435,20 +435,6 @@ panel, resize it from any edge or corner, and double-click the header to dock it again. The position persists across reloads. -- Give each Statens vegvesen camera its own ground height from Kartverket's - keyless point-height API (CC BY 4.0) instead of one flat 150 m prior, so - mountain-pass cameras no longer sit hundreds of metres below the terrain - where neither the point-height prior nor the ground snap resolves. About - 850 of ~855 cameras get a height; the rest keep the old prior. Requests - carry 20 points, the most the API answers from its 1 m terrain model, eight - at a time (~6 s cold). Cameras over water use the water surface rather than - the sea-floor depth the API returns there; for bridge cameras that is still - below the deck. Heights are cached for good and misses for 6 h, so a - refresh only asks about new or moved cameras. After a 429, a 5xx or a - network failure the lookup stops and backs off (Retry-After, else 10 min). - The lane waits at most 12 s from its start; a slower lookup fills the cache - for the next refresh. `CCTV_VEGVESEN_HEIGHTS=0` turns it off. Follow-up to - #929. - Add Norway's city bikes to the Bike Share layer: Oslo, Bergen and Trondheim Bysykkel (Urban Sharing), Kolumbus Bysykkel in Stavanger and Farte Bysykkel in Skien/Porsgrunn (Entur Mobility), and Bysykkel diff --git a/changelog.d/945-cctv-vegvesen-kartverket-heights.md b/changelog.d/945-cctv-vegvesen-kartverket-heights.md new file mode 100644 index 000000000..e300b702c --- /dev/null +++ b/changelog.d/945-cctv-vegvesen-kartverket-heights.md @@ -0,0 +1,14 @@ +- Give each Statens vegvesen camera its own ground height from Kartverket's + keyless point-height API (CC BY 4.0) instead of one flat 150 m prior, so + mountain-pass cameras no longer sit hundreds of metres below the terrain + where neither the point-height prior nor the ground snap resolves. About 850 + of ~855 cameras get a height; the rest keep the old prior. Requests carry 20 + points, the most the API answers from its 1 m terrain model, eight at a time + (~6 s cold). Cameras over water use the water surface rather than the + sea-floor depth the API returns there; for bridge cameras that is still + below the deck. Heights are cached for good and misses for 6 h, so a refresh + only asks about new or moved cameras. After a 429, a 5xx or a network + failure the lookup stops and backs off (Retry-After, else 10 min). The lane + waits at most 12 s from its start; a slower lookup fills the cache for the + next refresh. `CCTV_VEGVESEN_HEIGHTS=0` turns it off. Follow-up to #929 + (trygvehhh, #945). From 59db4ac22798a89e84a157ba2bf4665f6229f488 Mon Sep 17 00:00:00 2001 From: Trygve Hemminghytt <274629477+trygvehhh@users.noreply.github.com> Date: Sun, 18 Oct 2026 20:22:21 +0200 Subject: [PATCH 4/9] docs(cctv): state the measured cold lookup time and what a slow first run leaves --- DATA_SOURCES.md | 2 +- .../945-cctv-vegvesen-kartverket-heights.md | 19 ++++++++++--------- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/DATA_SOURCES.md b/DATA_SOURCES.md index b77ee5678..f2b9f8fb9 100644 --- a/DATA_SOURCES.md +++ b/DATA_SOURCES.md @@ -167,7 +167,7 @@ The optional Local ADS-B layer and the broadcast-FM receiver use radio you recei - **King County road cameras.** Keyless. The camera list is one public ArcGIS FeatureServer GeoJSON query (King County DOT Road Services, "King County and WSDOT Traffic Cameras"). The `outFields` list is explicit on purpose: the layer also carries asset-administration columns — including staff names — that are never fetched. Only live, active cameras with frames on the county's own `info.kingcounty.gov` image handler are registered (the feed writes `http://`; the handler serves HTTPS, so URLs are upgraded and pinned before registration). WSDOT-owned rows in the layer are skipped because their frames are WSDOT-hosted, outside the pinned image origin; partner-city cameras hosted on the county image handler (e.g. Newcastle) name their owner as the per-camera credit. The layer publishes no facing, so headings use the shared id-hash fallback at low confidence. Uniquely, the county publishes each camera's hardware `Manufacturer`/`Model` (Cohu 3950/3965, Bosch AUTODOME, Axis PTZ domes); both are carried on the source (and served by `/api/cctv/sources`) as `model` and `manufacturer` for identified-hardware consumers (vendor-generic strings such as `PTZ` name no model and are omitted). Frames are fetched at request time and never stored or redistributed. Disable with `CCTV_KINGCOUNTY_ENABLED=0`. - **WSDOT highway cameras.** Keyless. The camera list is the Esri JSON layer `data.wsdot.wa.gov/travelcenter/Cameras.json` — the same layer WSDOT's own Travel Center map consumes (~1,700 point features, EPSG:3857 geometry converted to WGS84 on load; the body carries Windows-1252 bytes and is decoded as such). Only frames on WSDOT's own `images.wsdot.wa.gov` host are registered — the catalog also lists ~70 partner cameras (Oregon DOT, City of Seattle, lodges, airports) whose own terms apply, and those are skipped rather than proxied. WSDOT's layer metadata marks the feed for "low volume" use (see WSDOT's [travel information disclaimer](https://wsdot.wa.gov/about/policies/travel-information-disclaimer)). The catalog is fetched once per CCTV catalog refresh (every 15 minutes), and a camera's frame is fetched only when that camera is viewed; frames are never stored or redistributed. The live feed's `CompassDirection` is a single cardinal (N/E/S/W), `B` (both directions — most rows), `O`, or null. A single cardinal gives a high-confidence heading (the diagonals NE/SE/SW/NW are mapped too should they appear); `B`, `O`, null, and any other code carry no single facing and use the shared id-hash fallback at low confidence. By default the 250 cameras nearest Seattle and Spokane load; `CCTV_WSDOT_MAX_SOURCES` changes the cap (8–1,200) and `CCTV_WSDOT_ENABLED=0` turns the pack off. A keyed WSDOT Traveler Information API serves the same population for higher-volume use and is deliberately not required here. - **Statens vegvesen road cameras.** Keyless. The camera list is one GeoJSON request to the OGC API Features view of the official DATEX 3.1 CCTV site table (`ogckart-sn1.atlas.vegvesen.no/ogc/features/v1/collections/datex_3_1:CctvSimple/items`), so no DATEX account is needed; frames are stills on `kamera.atlas.vegvesen.no/api/images/`, and only that exact path per camera is registered. Cameras the feed reports as faulty are dropped. About 135 working cameras also publish live HLS (`kamera.vegvesen.no/public//manifest.m3u8`, MPEG-TS); those play as live video through the bounded in-memory HLS relay, with the still as the snapshot fallback, and only that exact manifest path per camera is registered. `CCTV_VEGVESEN_VIDEO=0` keeps every camera on stills. `orientationDescription` names the place a camera faces, not a bearing, so it goes into the label and headings use the id-hash fallback at low confidence. By default every live camera loads (~850); `CCTV_VEGVESEN_MAX_SOURCES` (8–1000) lowers the cap, keeping the cameras nearest Oslo, Bergen, Stavanger, Trondheim, Kristiansand, Bodø and Tromsø and `CCTV_VEGVESEN_ENABLED=0` turns the pack off. Frames are fetched at request time and never stored or redistributed. The NLOD attribution is registered in the Data attribution popover. -- **Kartverket ground heights for Statens vegvesen cameras.** The camera feed's points are 2D, so the loader asks Kartverket's keyless point-height API (`ws.geonorge.no/hoydedata/v1/punkt`, CC BY 4.0) for the terrain height at each camera. Requests carry 20 points: the API accepts 50, but above ~20 it answers the extra points from contour lines (`hoydekurver`, metres to ~20 m off) or not at all. Eight requests run at a time, about 36 for the whole country (~6 s). Answers are matched by the coordinates the API echoes and recorded only for points that request asked about. Over water the API returns the sea-floor depth (`dybdekurver` or `Havflate`), which is replaced by 0 m, the water surface; for the ~14 bridge cameras that is below the deck, but much closer than the sea floor or the flat prior. Heights are cached in memory by position for the life of the server, and points with no height (a few ferry quays and bridges) for 6 h, so the 15-minute catalog refresh only asks about new or moved cameras. The first 429, 5xx or network failure stops the lookup, with no retries, and no new lookup starts until the backoff ends (Retry-After when sent, else 10 min; bounded to 1 min–1 h). The lane waits at most 12 s, counted from the start of the loader; a slower lookup keeps filling the cache for the next refresh. Only coordinates are sent, with the same User-Agent as the other CCTV requests. `CCTV_VEGVESEN_HEIGHTS=0` turns the lookup off. The Kartverket credit is shown beside the Statens vegvesen entry in the Data attribution popover. +- **Kartverket ground heights for Statens vegvesen cameras.** The camera feed's points are 2D, so the loader asks Kartverket's keyless point-height API (`ws.geonorge.no/hoydedata/v1/punkt`, CC BY 4.0) for the terrain height at each camera. Requests carry 20 points: the API accepts 50, but above ~20 it answers the extra points from contour lines (`hoydekurver`, metres to ~20 m off) or not at all. Eight requests run at a time, about 36 for the whole country: a cold lookup took 6–13 s in testing (single requests 0.6–9.5 s). Answers are matched by the coordinates the API echoes and recorded only for points that request asked about. Over water the API returns the sea-floor depth (`dybdekurver` or `Havflate`), which is replaced by 0 m, the water surface; for the ~14 bridge cameras that is below the deck, but much closer than the sea floor or the flat prior. Heights are cached in memory by position for the life of the server, and points with no height (a few ferry quays and bridges) for 6 h, so the 15-minute catalog refresh only asks about new or moved cameras. The first 429, 5xx or network failure stops the lookup, with no retries, and no new lookup starts until the backoff ends (Retry-After when sent, else 10 min; bounded to 1 min–1 h). The lane waits at most 12 s, counted from the start of the loader; a slower lookup keeps filling the cache, and the cameras it had not reached keep the flat prior until the next 15-minute refresh (about 190 of 865 on one slow first run). Only coordinates are sent, with the same User-Agent as the other CCTV requests. `CCTV_VEGVESEN_HEIGHTS=0` turns the lookup off. The Kartverket credit is shown beside the Statens vegvesen entry in the Data attribution popover. - **NYC DOT Traffic Management Center cameras.** Off by default. NYC DOT's [developer data feeds page](https://www.nyc.gov/html/dot/html/about/datafeeds.shtml) asks developers who want the camera feed to contact `tmcdot@dot.nyc.gov` and sign a data-sharing agreement. This project does not hold one that covers downstream installations, so the pack loads only when an operator who holds their own agreement sets `CCTV_NYC_ENABLED=1`; with the flag unset, empty or any other value, the server makes no request to NYC DOT. The endpoint needs no key. The camera list is the public `webcams.nyctmc.org/api/cameras` catalog (~980 cameras across the five boroughs); frames are stills at `webcams.nyctmc.org/api/cameras//image`, synthesized from a validated UUID rather than taken from the payload, and only that host is registered. Cameras the feed reports as offline are dropped. The catalog publishes no camera facing, so headings use the shared id-hash fallback at low confidence and are corrected with the in-app calibration gizmo. Once enabled, the 250 cameras nearest the borough cores load; `CCTV_NYC_MAX_SOURCES` (8–400) changes the cap. Frames are fetched at request time and never stored or redistributed. NYC DOT asks every application that uses its data to carry its disclaimer; it is registered verbatim in the Data attribution popover. - **City of Toronto RESCU cameras.** Keyless. The camera list is the City's public `tmcearthcameras.json` on `opendata.toronto.ca` (dataset "Traffic Cameras", Transportation Services), served as JSONP and unwrapped with an anchored pattern. This is the municipal network the Ontario 511 pack does not carry: downtown arterials plus the Gardiner, DVP, Allen and Lake Shore. The list publishes no frame URL, so frame URLs are BUILT from the documented `CameraImages/loc####.jpg` convention and a strictly-digits camera number, pinning the frame proxy to the portal origin by construction; the list fetch refuses redirects and bounds the body it buffers. Frames are fetched at request time and never stored or redistributed. The dataset publishes no camera facing: the `D1`-`D4` fields are the directions of the *comparison* images (static reference stills captured in 2016 under a different path), not a bearing for the live pan-tilt-zoom frame, so headings use the shared id-hash fallback at low confidence and are corrected with the in-app calibration gizmo. A camera that is temporarily out of service still answers 200 with a byte-identical "Image Unavailable" placeholder rather than an error (about 6% of the network at any moment), so such a camera reads as healthy; the frame it shows is the agency's, not a synthesized one. The licence requires the attribution statement above. Disable with `CCTV_TORONTO_ENABLED=0`. - **Camera frame content.** Public camera frames can contain people, vehicles and license plates, and the camera layer does nothing with them beyond putting them on the map. A successful upstream response is relayed as the provider served it: nothing in the camera pipeline blurs, redacts, enhances or restores it, and nothing there runs face, plate or object detection or otherwise recognises or classifies what is in the picture. (The ALPR layer described below maps where plate-reader cameras are, from locations contributors tagged in OpenStreetMap; it reads nothing from any camera frame.) When an upstream has no frame for a camera, what you see instead is a Google Street View still or the app's own placeholder card; the panel's source badge names which of them you are looking at, and a refresh that fails after a frame has arrived leaves the last good picture on screen rather than replacing it. The browser resamples each frame for display: onto a fixed 1920x1080 surface for the camera's projection plane in the scene, and onto a 192x108 canvas for the small ambient card. That scaling is the only change made to the picture, and the only pixels the camera layer reads back are a 64x36 downscale hashed to tell whether the picture has changed. Live streams are piped through; the still path reads the image into memory to pass it on — TxDOT wraps its JPEG in a JSON envelope, which is decoded there — and no frame is written to disk (the disk cache, `.gev-cache/`, holds feed JSON for other layers and never camera imagery). Outside that pipeline, the voice assistant is the one feature that sends imagery anywhere: when it answers a question about what is in view it may attach a downscaled screenshot of the rendered scene — including any camera plane on screen — to the request it sends to the voice model. Frames are fetched only from server-registered camera URLs — a client cannot hand the proxy a URL (see [SECURITY.md](SECURITY.md)). What the app does stays inside the line README draws: no named-person search, no face recognition, no tracking individuals. diff --git a/changelog.d/945-cctv-vegvesen-kartverket-heights.md b/changelog.d/945-cctv-vegvesen-kartverket-heights.md index e300b702c..c6c4b41d9 100644 --- a/changelog.d/945-cctv-vegvesen-kartverket-heights.md +++ b/changelog.d/945-cctv-vegvesen-kartverket-heights.md @@ -3,12 +3,13 @@ mountain-pass cameras no longer sit hundreds of metres below the terrain where neither the point-height prior nor the ground snap resolves. About 850 of ~855 cameras get a height; the rest keep the old prior. Requests carry 20 - points, the most the API answers from its 1 m terrain model, eight at a time - (~6 s cold). Cameras over water use the water surface rather than the - sea-floor depth the API returns there; for bridge cameras that is still - below the deck. Heights are cached for good and misses for 6 h, so a refresh - only asks about new or moved cameras. After a 429, a 5xx or a network - failure the lookup stops and backs off (Retry-After, else 10 min). The lane - waits at most 12 s from its start; a slower lookup fills the cache for the - next refresh. `CCTV_VEGVESEN_HEIGHTS=0` turns it off. Follow-up to #929 - (trygvehhh, #945). + points, the most the API answers from its 1 m terrain model, eight at a + time; a cold lookup takes about 6–13 s. Cameras over water use the water + surface rather than the sea-floor depth the API returns there; for bridge + cameras that is still below the deck. Heights are cached for good and misses + for 6 h, so a refresh only asks about new or moved cameras. After a 429, a + 5xx or a network failure the lookup stops and backs off (Retry-After, else + 10 min). The lane waits at most 12 s from its start; a slower first lookup + leaves the cameras it had not reached on the flat prior until the next + 15-minute refresh, and keeps filling the cache meanwhile. + `CCTV_VEGVESEN_HEIGHTS=0` turns it off. Follow-up to #929 (trygvehhh, #945). From 3792d7377e2aedb3138b86d313b0512442cfcd47 Mon Sep 17 00:00:00 2001 From: yaka0007 Date: Mon, 19 Oct 2026 01:12:18 -0400 Subject: [PATCH 5/9] fix(cctv): harden Tarktee DATEX catalog fetches and dedupe before cap --- server/providers/cctv/constants.js | 5 ++ server/providers/cctv/sources.js | 58 ++++++++++-- src/data/cctvEstonia.test.mjs | 139 +++++++++++++++++++++++++++++ 3 files changed, 194 insertions(+), 8 deletions(-) diff --git a/server/providers/cctv/constants.js b/server/providers/cctv/constants.js index 20b963770..9bdb5e511 100644 --- a/server/providers/cctv/constants.js +++ b/server/providers/cctv/constants.js @@ -188,6 +188,11 @@ export const TARKTEE_IMAGES_URL = 'https://tarktee.transpordiamet.ee/api/v1/datex/roadCameraImages'; export const TARKTEE_IMAGE_ORIGIN = 'https://tarktee.transpordiamet.ee/images/'; export const DEFAULT_TARKTEE_MAX_SOURCES = 179; +/** Hard ceiling per Tarktee DATEX XML document (~120–165 KB live). */ +export const TARKTEE_MAX_CATALOG_BYTES = 4 * 1024 * 1024; +/** Two DATEX XML documents download concurrently from the same origin (~5.3 s + * baseline) alongside the other packs, so the 8 s default can abort mid-stream. */ +export const TARKTEE_FETCH_TIMEOUT_MS = 15_000; export const TARKTEE_ANCHORS = [ { lat: 59.437, lon: 24.753 }, // Tallinn { lat: 58.378, lon: 26.729 }, // Tartu diff --git a/server/providers/cctv/sources.js b/server/providers/cctv/sources.js index c3d11e752..cba857565 100644 --- a/server/providers/cctv/sources.js +++ b/server/providers/cctv/sources.js @@ -42,6 +42,8 @@ import { TARKTEE_IMAGES_URL, TARKTEE_IMAGE_ORIGIN, DEFAULT_TARKTEE_MAX_SOURCES, + TARKTEE_MAX_CATALOG_BYTES, + TARKTEE_FETCH_TIMEOUT_MS, TARKTEE_ANCHORS, DEFAULT_WARENDORF_SOURCE_FILE, WARENDORF_IMAGE_ORIGINS, @@ -1194,25 +1196,62 @@ export async function loadTarkteeSourcesFromDatex() { const [locResp, imgResp] = await Promise.all([ fetch(TARKTEE_LOCATIONS_URL, { headers: { Accept: 'application/xml,text/xml,*/*' }, - signal: AbortSignal.timeout(CCTV_SOURCE_FETCH_TIMEOUT_MS), + redirect: 'manual', + signal: AbortSignal.timeout(TARKTEE_FETCH_TIMEOUT_MS), }), fetch(TARKTEE_IMAGES_URL, { headers: { Accept: 'application/xml,text/xml,*/*' }, - signal: AbortSignal.timeout(CCTV_SOURCE_FETCH_TIMEOUT_MS), + redirect: 'manual', + signal: AbortSignal.timeout(TARKTEE_FETCH_TIMEOUT_MS), }), ]); + // Both responses are fetched in parallel; if either is rejected or overflows + // its byte cap, both bodies must be released so neither transport leaks. + const discardBoth = async () => { + await Promise.allSettled([ + locResp?.body?.cancel(), + imgResp?.body?.cancel(), + ]); + return []; + }; + if ( + (locResp.status >= 300 && locResp.status < 400) || + (imgResp.status >= 300 && imgResp.status < 400) + ) { + console.warn( + '[CCTV] Tarktee DATEX catalog redirected; redirects are not followed', + ); + return discardBoth(); + } if (!locResp.ok) { console.warn('[CCTV] Tarktee locations download failed:', locResp.status); - return []; + return discardBoth(); } if (!imgResp.ok) { console.warn('[CCTV] Tarktee images download failed:', imgResp.status); - return []; + return discardBoth(); + } + let locXml; + let imgXml; + const readAbort = new AbortController(); + try { + [locXml, imgXml] = await Promise.all([ + readResponseTextCapped( + locResp, + TARKTEE_MAX_CATALOG_BYTES, + readAbort.signal, + ), + readResponseTextCapped( + imgResp, + TARKTEE_MAX_CATALOG_BYTES, + readAbort.signal, + ), + ]); + } catch (error) { + readAbort.abort(); + await discardBoth(); + throw error; } - const [locXml, imgXml] = await Promise.all([ - locResp.text(), - imgResp.text(), - ]); const locations = parseTarkteeDatexLocations(locXml); const images = parseTarkteeDatexImages(imgXml); if (!locations.size || !images.size) { @@ -1224,6 +1263,7 @@ export async function loadTarkteeSourcesFromDatex() { } const cameras = []; + const seen = new Set(); for (const [locationId, loc] of locations.entries()) { const imageUrl = images.get(locationId); if (!imageUrl) continue; @@ -1235,6 +1275,8 @@ export async function loadTarkteeSourcesFromDatex() { const cameraId = numMatch ? `ee-tarktee-${numMatch[1]}` : `ee-tarktee-${locationId}`; + if (seen.has(cameraId)) continue; + seen.add(cameraId); cameras.push({ id: cameraId, name: loc.name, diff --git a/src/data/cctvEstonia.test.mjs b/src/data/cctvEstonia.test.mjs index 7c0ebaba1..a6385854a 100644 --- a/src/data/cctvEstonia.test.mjs +++ b/src/data/cctvEstonia.test.mjs @@ -1,11 +1,23 @@ import { test } from 'node:test'; import assert from 'node:assert/strict'; +import { + TARKTEE_IMAGES_URL, + TARKTEE_LOCATIONS_URL, + TARKTEE_MAX_CATALOG_BYTES, +} from '../../server/providers/cctv/constants.js'; import { loadTallinnSourcesFromCatalog, + loadTarkteeSourcesFromDatex, parseTarkteeDatexImages, parseTarkteeDatexLocations, } from '../../server/providers/cctv/sources.js'; +/** Silence the loaders' progress and failure logging. */ +function quiet(t) { + t.mock.method(console, 'log', () => {}); + t.mock.method(console, 'warn', () => {}); +} + const SAMPLE_LOCATIONS_XML = ` @@ -84,3 +96,130 @@ test('Tarktee DATEX image parser rejects non-official hosts', () => { const images = parseTarkteeDatexImages(xml); assert.equal(images.size, 0); }); + +test('Tarktee loader deduplicates cameras sharing an image number before the cap', async (t) => { + quiet(t); + const dupLocationsXml = ` + + + + + Tallinn Ringtee + 59.437024.7530 + + + Tallinn Ringtee duplicate + 59.437124.7531 + + + +`; + const dupImagesXml = ` + + + https://tarktee.transpordiamet.ee/images/42/42_1.jpg + https://tarktee.transpordiamet.ee/images/42/42_2.jpg + +`; + + const seen = []; + t.mock.method(globalThis, 'fetch', async (url, init) => { + seen.push([String(url), init?.redirect]); + const body = + String(url) === TARKTEE_LOCATIONS_URL ? dupLocationsXml : dupImagesXml; + return new Response(body, { + status: 200, + headers: { 'Content-Type': 'application/xml' }, + }); + }); + + const cameras = await loadTarkteeSourcesFromDatex(); + assert.deepEqual(seen, [ + [TARKTEE_LOCATIONS_URL, 'manual'], + [TARKTEE_IMAGES_URL, 'manual'], + ]); + assert.equal(cameras.length, 1); + assert.equal(cameras[0].id, 'ee-tarktee-42'); + assert.equal(cameras[0].name, 'Tallinn Ringtee'); +}); + +test('Tarktee loader refuses a redirect and cancels both response bodies', async (t) => { + quiet(t); + const cancelled = { locations: false, images: false }; + t.mock.method(globalThis, 'fetch', async (url) => { + const isLocations = String(url) === TARKTEE_LOCATIONS_URL; + const body = new ReadableStream({ + start(controller) { + controller.enqueue(new TextEncoder().encode('')); + }, + cancel() { + if (isLocations) cancelled.locations = true; + else cancelled.images = true; + }, + }); + return new Response(body, { + status: isLocations ? 302 : 200, + headers: isLocations ? { location: 'https://evil.example/loc.xml' } : {}, + }); + }); + + assert.deepEqual(await loadTarkteeSourcesFromDatex(), []); + assert.equal(cancelled.locations, true, 'redirected body is cancelled'); + assert.equal(cancelled.images, true, 'sibling 200 body is also cancelled'); +}); + +test('Tarktee loader cancels the sibling body when one feed fails or overflows the byte cap', async (t) => { + quiet(t); + let siblingCancelled = false; + t.mock.method(globalThis, 'fetch', async (url) => { + const isLocations = String(url) === TARKTEE_LOCATIONS_URL; + if (isLocations) { + return new Response('', { status: 503 }); + } + const body = new ReadableStream({ + start(controller) { + controller.enqueue(new TextEncoder().encode(SAMPLE_IMAGES_XML)); + }, + cancel() { + siblingCancelled = true; + }, + }); + return new Response(body, { status: 200 }); + }); + + assert.deepEqual(await loadTarkteeSourcesFromDatex(), []); + assert.equal( + siblingCancelled, + true, + 'sibling 200 stream is cancelled on 503', + ); + + let overflowSiblingCancelled = false; + t.mock.method(globalThis, 'fetch', async (url) => { + const isLocations = String(url) === TARKTEE_LOCATIONS_URL; + if (isLocations) { + const padded = + SAMPLE_LOCATIONS_XML + ' '.repeat(TARKTEE_MAX_CATALOG_BYTES); + return new Response(padded, { + status: 200, + headers: { 'Content-Length': String(padded.length) }, + }); + } + const body = new ReadableStream({ + start(controller) { + controller.enqueue(new TextEncoder().encode(SAMPLE_IMAGES_XML)); + }, + cancel() { + overflowSiblingCancelled = true; + }, + }); + return new Response(body, { status: 200 }); + }); + + assert.deepEqual(await loadTarkteeSourcesFromDatex(), []); + assert.equal( + overflowSiblingCancelled, + true, + 'sibling stream is cancelled when the other feed exceeds the byte cap', + ); +}); From 9b400622af0cb86d957fefe8d9e642efb6c69ec0 Mon Sep 17 00:00:00 2001 From: yaka0007 Date: Mon, 19 Oct 2026 01:13:30 -0400 Subject: [PATCH 6/9] docs(changelog): add fragment for #1233 --- changelog.d/1233-tarktee-datex-catalog-hardening.md | 4 ++++ 1 file changed, 4 insertions(+) create mode 100644 changelog.d/1233-tarktee-datex-catalog-hardening.md diff --git a/changelog.d/1233-tarktee-datex-catalog-hardening.md b/changelog.d/1233-tarktee-datex-catalog-hardening.md new file mode 100644 index 000000000..11b9641e2 --- /dev/null +++ b/changelog.d/1233-tarktee-datex-catalog-hardening.md @@ -0,0 +1,4 @@ +- Harden the Tarktee (Estonia) DATEX2 camera catalog fetches: refuse redirects, + cancel both response streams when either request fails or overflows the 4 MB + byte cap, extend the dual-XML fetch deadline to 15 s, and deduplicate camera + IDs before per-pack prioritization (yaka0007, #1233). From c808bfe609b491e33fea9da4a0ed22f72cb8f7a3 Mon Sep 17 00:00:00 2001 From: Said Turksever Date: Mon, 19 Oct 2026 17:03:50 +0100 Subject: [PATCH 7/9] fix(cctv): harden the Kartverket height lookup for Vegvesen cameras - Default backoff is 30 min, longer than the 15-minute catalog refresh, so a failing Kartverket costs one attempt per backoff, not per refresh. - A point a 200 answer leaves out is cached as no height (6 h TTL) instead of being asked again, and waited on, every refresh. - Only 408, 429 and 5xx stop the lookup; any other 4xx marks that request's points as no height and the lookup carries on. - Each request has its own 10 s timeout inside the shared 30 s deadline; a lookup cut short only by the deadline resumes without a backoff. - The lane waits only on a cold cache (>10% of cameras missing); a few new cameras or expired misses are looked up in the background. - applyVegvesenGroundHeights skips cameras without finite coordinates, so it really never throws. - clearKartverketHeightCache bumps a generation, so a lookup started before a clear cannot write to the cache, backoff or pending slot. - Credit and DATA_SOURCES note the modification (water set to 0 m, rounded to 0.1 m) as CC BY 4.0 asks; docs match the stop rules. - Tests for the deadline being shared, per-request timeouts, 503, oversized and non-JSON bodies, 4xx, left-out points, Retry-After end to end, overlapping refreshes, the warm-cache wait, bad coordinates, the cache-clear generation and the loader's wait budget. Co-Authored-By: Claude Opus 5.5 --- DATA_SOURCES.md | 4 +- .../945-cctv-vegvesen-kartverket-heights.md | 12 +- server/providers/cctv/constants.js | 22 +- server/providers/cctv/sources.js | 101 ++++-- src/data/cctvVegvesen.test.mjs | 311 +++++++++++++++++- src/data/dataCredits.js | 3 +- 6 files changed, 399 insertions(+), 54 deletions(-) diff --git a/DATA_SOURCES.md b/DATA_SOURCES.md index 43d92b4ca..c99ec28d1 100644 --- a/DATA_SOURCES.md +++ b/DATA_SOURCES.md @@ -64,7 +64,7 @@ How to read this: | **King County Road Services** (`services.arcgis.com`, King County GIS) | CCTV camera catalog + frames, unincorporated King County WA road network | [KCGIS Center terms](https://kingcounty.gov/en/dept/kcit/data-information-services/gis-center/about/terms-conditions-copyrights) — attribution required; public ArcGIS layer | "King County Road Services" (partner-city cameras name their owner) | | **WSDOT** (`data.wsdot.wa.gov`) | CCTV camera catalog + frames, Washington State highways | [WSDOT travel information disclaimer](https://wsdot.wa.gov/about/policies/travel-information-disclaimer) — public "low volume" feed; catalog cached, frames on WSDOT's own image host only | "Washington State Department of Transportation (Travel Center)" (courtesy) | | **Statens vegvesen** (catalog `ogckart-sn1.atlas.vegvesen.no`, frames `kamera.atlas.vegvesen.no`, HLS `kamera.vegvesen.no`) | CCTV camera catalog + frames + live HLS video, Norway road cameras | [NLOD 2.0](https://data.norge.no/nlod/en/2.0) — attribution REQUIRED | "Contains data under the Norwegian licence for Open Government data (NLOD) distributed by Statens vegvesen" | -| **Kartverket** (`ws.geonorge.no/hoydedata/v1/punkt`) | Terrain heights (NN2000) used as the ground prior for Statens vegvesen cameras | [CC BY 4.0](https://creativecommons.org/licenses/by/4.0/) — attribution REQUIRED | "© Kartverket" | +| **Kartverket** (`ws.geonorge.no/hoydedata/v1/punkt`) | Terrain heights (NN2000) used as the ground prior for Statens vegvesen cameras | [CC BY 4.0](https://creativecommons.org/licenses/by/4.0/) — attribution REQUIRED | "© Kartverket", modified (heights over water set to 0 m, rounded to 0.1 m) | | **NYC DOT Traffic Management Center** (`webcams.nyctmc.org`) | CCTV camera catalog + frames, New York City five boroughs | NYC DOT requires developers to sign a [data-sharing agreement](https://www.nyc.gov/html/dot/html/about/datafeeds.shtml) for the camera feed. **Off by default**; enable only under your own agreement | "NYC DOT Traffic Management Center" + NYC DOT's required application disclaimer (in the Data attribution popover) | | **City of Toronto Open Data** (`opendata.toronto.ca`) | CCTV camera catalog + frames, Toronto city streets | [Open Government Licence – Toronto](https://www.toronto.ca/city-government/data-research-maps/open-data/open-data-licence/) — attribution REQUIRED | "Contains information licensed under the Open Government Licence – Toronto" | | **GBFS (Lyft / BCycle / Social Bicycles / PBSC)** | Bikeshare availability | Per-feed (attribution-only) | Credit the operator (e.g. Austin BCycle) + its `license_url` | @@ -169,7 +169,7 @@ The optional Local ADS-B layer and the broadcast-FM receiver use radio you recei - **King County road cameras.** Keyless. The camera list is one public ArcGIS FeatureServer GeoJSON query (King County DOT Road Services, "King County and WSDOT Traffic Cameras"). The `outFields` list is explicit on purpose: the layer also carries asset-administration columns — including staff names — that are never fetched. Only live, active cameras with frames on the county's own `info.kingcounty.gov` image handler are registered (the feed writes `http://`; the handler serves HTTPS, so URLs are upgraded and pinned before registration). WSDOT-owned rows in the layer are skipped because their frames are WSDOT-hosted, outside the pinned image origin; partner-city cameras hosted on the county image handler (e.g. Newcastle) name their owner as the per-camera credit. The layer publishes no facing, so headings use the shared id-hash fallback at low confidence. Uniquely, the county publishes each camera's hardware `Manufacturer`/`Model` (Cohu 3950/3965, Bosch AUTODOME, Axis PTZ domes); both are carried on the source (and served by `/api/cctv/sources`) as `model` and `manufacturer` for identified-hardware consumers (vendor-generic strings such as `PTZ` name no model and are omitted). Frames are fetched at request time and never stored or redistributed. Disable with `CCTV_KINGCOUNTY_ENABLED=0`. - **WSDOT highway cameras.** Keyless. The camera list is the Esri JSON layer `data.wsdot.wa.gov/travelcenter/Cameras.json` — the same layer WSDOT's own Travel Center map consumes (~1,700 point features, EPSG:3857 geometry converted to WGS84 on load; the body carries Windows-1252 bytes and is decoded as such). Only frames on WSDOT's own `images.wsdot.wa.gov` host are registered — the catalog also lists ~70 partner cameras (Oregon DOT, City of Seattle, lodges, airports) whose own terms apply, and those are skipped rather than proxied. WSDOT's layer metadata marks the feed for "low volume" use (see WSDOT's [travel information disclaimer](https://wsdot.wa.gov/about/policies/travel-information-disclaimer)). The catalog is fetched once per CCTV catalog refresh (every 15 minutes), and a camera's frame is fetched only when that camera is viewed; frames are never stored or redistributed. The live feed's `CompassDirection` is a single cardinal (N/E/S/W), `B` (both directions — most rows), `O`, or null. A single cardinal gives a high-confidence heading (the diagonals NE/SE/SW/NW are mapped too should they appear); `B`, `O`, null, and any other code carry no single facing and use the shared id-hash fallback at low confidence. By default the 250 cameras nearest Seattle and Spokane load; `CCTV_WSDOT_MAX_SOURCES` changes the cap (8–1,200) and `CCTV_WSDOT_ENABLED=0` turns the pack off. A keyed WSDOT Traveler Information API serves the same population for higher-volume use and is deliberately not required here. - **Statens vegvesen road cameras.** Keyless. The camera list is one GeoJSON request to the OGC API Features view of the official DATEX 3.1 CCTV site table (`ogckart-sn1.atlas.vegvesen.no/ogc/features/v1/collections/datex_3_1:CctvSimple/items`), so no DATEX account is needed; frames are stills on `kamera.atlas.vegvesen.no/api/images/`, and only that exact path per camera is registered. Cameras the feed reports as faulty are dropped. About 135 working cameras also publish live HLS (`kamera.vegvesen.no/public//manifest.m3u8`, MPEG-TS); those play as live video through the bounded in-memory HLS relay, with the still as the snapshot fallback, and only that exact manifest path per camera is registered. `CCTV_VEGVESEN_VIDEO=0` keeps every camera on stills. `orientationDescription` names the place a camera faces, not a bearing, so it goes into the label and headings use the id-hash fallback at low confidence. By default every live camera loads (~850); `CCTV_VEGVESEN_MAX_SOURCES` (8–1000) lowers the cap, keeping the cameras nearest Oslo, Bergen, Stavanger, Trondheim, Kristiansand, Bodø and Tromsø and `CCTV_VEGVESEN_ENABLED=0` turns the pack off. Frames are fetched at request time and never stored or redistributed. The NLOD attribution is registered in the Data attribution popover. -- **Kartverket ground heights for Statens vegvesen cameras.** The camera feed's points are 2D, so the loader asks Kartverket's keyless point-height API (`ws.geonorge.no/hoydedata/v1/punkt`, CC BY 4.0) for the terrain height at each camera. Requests carry 20 points: the API accepts 50, but above ~20 it answers the extra points from contour lines (`hoydekurver`, metres to ~20 m off) or not at all. Eight requests run at a time, about 36 for the whole country: a cold lookup took 6–13 s in testing (single requests 0.6–9.5 s). Answers are matched by the coordinates the API echoes and recorded only for points that request asked about. Over water the API returns the sea-floor depth (`dybdekurver` or `Havflate`), which is replaced by 0 m, the water surface; for the ~14 bridge cameras that is below the deck, but much closer than the sea floor or the flat prior. Heights are cached in memory by position for the life of the server, and points with no height (a few ferry quays and bridges) for 6 h, so the 15-minute catalog refresh only asks about new or moved cameras. The first 429, 5xx or network failure stops the lookup, with no retries, and no new lookup starts until the backoff ends (Retry-After when sent, else 10 min; bounded to 1 min–1 h). The lane waits at most 12 s, counted from the start of the loader; a slower lookup keeps filling the cache, and the cameras it had not reached keep the flat prior until the next 15-minute refresh (about 190 of 865 on one slow first run). Only coordinates are sent, with the same User-Agent as the other CCTV requests. `CCTV_VEGVESEN_HEIGHTS=0` turns the lookup off. The Kartverket credit is shown beside the Statens vegvesen entry in the Data attribution popover. +- **Kartverket ground heights for Statens vegvesen cameras.** The camera feed's points are 2D, so the loader asks Kartverket's keyless point-height API (`ws.geonorge.no/hoydedata/v1/punkt`, CC BY 4.0) for the terrain height at each camera. Requests carry 20 points: the API accepts 50, but above ~20 it answers the extra points from contour lines (`hoydekurver`, metres to ~20 m off) or not at all. Eight requests run at a time, about 36 for the whole country: a cold lookup took 6–13 s in testing (single requests 0.6–9.5 s). Answers are matched by the coordinates the API echoes and recorded only for points that request asked about. Over water the API returns the sea-floor depth (`dybdekurver` or `Havflate`), which is replaced by 0 m, the water surface; for the ~14 bridge cameras that is below the deck, but much closer than the sea floor or the flat prior. Heights are cached in memory by position for the life of the server, and points with no height (a few ferry quays and bridges, or a point a 200 answer leaves out) for 6 h, so the 15-minute catalog refresh only asks about new or moved cameras. The first failing request (a 408, 429 or 5xx, a network error, a request over 10 s, or an unreadable or oversized body) stops the lookup, with no retries, and no new lookup starts until the backoff ends (Retry-After when sent, else 30 min, longer than the refresh; bounded to 1 min–1 h). Any other 4xx marks that request's points as having no height and the lookup carries on. The whole lookup has a 30 s deadline; a lookup cut short by it resumes on the next refresh without a backoff. The lane waits at most 12 s, counted from the start of the loader, and only when more than 10% of the cameras have no answer yet; a few new cameras or expired misses are looked up in the background. A slower lookup keeps filling the cache, and the cameras it had not reached keep the flat prior until the next 15-minute refresh (about 190 of 865 on one slow first run). Only coordinates are sent, with the same User-Agent as the other CCTV requests. `CCTV_VEGVESEN_HEIGHTS=0` turns the lookup off. The Kartverket credit, with a note that heights over water are set to 0 m and heights are rounded to 0.1 m, is shown beside the Statens vegvesen entry in the Data attribution popover. - **NYC DOT Traffic Management Center cameras.** Off by default. NYC DOT's [developer data feeds page](https://www.nyc.gov/html/dot/html/about/datafeeds.shtml) asks developers who want the camera feed to contact `tmcdot@dot.nyc.gov` and sign a data-sharing agreement. This project does not hold one that covers downstream installations, so the pack loads only when an operator who holds their own agreement sets `CCTV_NYC_ENABLED=1`; with the flag unset, empty or any other value, the server makes no request to NYC DOT. The endpoint needs no key. The camera list is the public `webcams.nyctmc.org/api/cameras` catalog (~980 cameras across the five boroughs); frames are stills at `webcams.nyctmc.org/api/cameras//image`, synthesized from a validated UUID rather than taken from the payload, and only that host is registered. Cameras the feed reports as offline are dropped. The catalog publishes no camera facing, so headings use the shared id-hash fallback at low confidence and are corrected with the in-app calibration gizmo. Once enabled, the 250 cameras nearest the borough cores load; `CCTV_NYC_MAX_SOURCES` (8–400) changes the cap. Frames are fetched at request time and never stored or redistributed. NYC DOT asks every application that uses its data to carry its disclaimer; it is registered verbatim in the Data attribution popover. - **City of Toronto RESCU cameras.** Keyless. The camera list is the City's public `tmcearthcameras.json` on `opendata.toronto.ca` (dataset "Traffic Cameras", Transportation Services), served as JSONP and unwrapped with an anchored pattern. This is the municipal network the Ontario 511 pack does not carry: downtown arterials plus the Gardiner, DVP, Allen and Lake Shore. The list publishes no frame URL, so frame URLs are BUILT from the documented `CameraImages/loc####.jpg` convention and a strictly-digits camera number, pinning the frame proxy to the portal origin by construction; the list fetch refuses redirects and bounds the body it buffers. Frames are fetched at request time and never stored or redistributed. The dataset publishes no camera facing: the `D1`-`D4` fields are the directions of the *comparison* images (static reference stills captured in 2016 under a different path), not a bearing for the live pan-tilt-zoom frame, so headings use the shared id-hash fallback at low confidence and are corrected with the in-app calibration gizmo. A camera that is temporarily out of service still answers 200 with a byte-identical "Image Unavailable" placeholder rather than an error (about 6% of the network at any moment), so such a camera reads as healthy; the frame it shows is the agency's, not a synthesized one. The licence requires the attribution statement above. Disable with `CCTV_TORONTO_ENABLED=0`. - **Camera frame content.** Public camera frames can contain people, vehicles and license plates, and the camera layer does nothing with them beyond putting them on the map. A successful upstream response is relayed as the provider served it: nothing in the camera pipeline blurs, redacts, enhances or restores it, and nothing there runs face, plate or object detection or otherwise recognises or classifies what is in the picture. (The ALPR layer described below maps where plate-reader cameras are, from locations contributors tagged in OpenStreetMap; it reads nothing from any camera frame.) When an upstream has no frame for a camera, what you see instead is a Google Street View still or the app's own placeholder card; the panel's source badge names which of them you are looking at, and a refresh that fails after a frame has arrived leaves the last good picture on screen rather than replacing it. The browser resamples each frame for display: onto a fixed 1920x1080 surface for the camera's projection plane in the scene, and onto a 192x108 canvas for the small ambient card. That scaling is the only change made to the picture, and the only pixels the camera layer reads back are a 64x36 downscale hashed to tell whether the picture has changed. Live streams are piped through; the still path reads the image into memory to pass it on — TxDOT wraps its JPEG in a JSON envelope, which is decoded there — and no frame is written to disk (the disk cache, `.gev-cache/`, holds feed JSON for other layers and never camera imagery). Outside that pipeline, the voice assistant is the one feature that sends imagery anywhere: when it answers a question about what is in view it may attach a downscaled screenshot of the rendered scene — including any camera plane on screen — to the request it sends to the voice model. Frames are fetched only from server-registered camera URLs — a client cannot hand the proxy a URL (see [SECURITY.md](SECURITY.md)). What the app does stays inside the line README draws: no named-person search, no face recognition, no tracking individuals. diff --git a/changelog.d/945-cctv-vegvesen-kartverket-heights.md b/changelog.d/945-cctv-vegvesen-kartverket-heights.md index c6c4b41d9..1572c0316 100644 --- a/changelog.d/945-cctv-vegvesen-kartverket-heights.md +++ b/changelog.d/945-cctv-vegvesen-kartverket-heights.md @@ -7,9 +7,11 @@ time; a cold lookup takes about 6–13 s. Cameras over water use the water surface rather than the sea-floor depth the API returns there; for bridge cameras that is still below the deck. Heights are cached for good and misses - for 6 h, so a refresh only asks about new or moved cameras. After a 429, a - 5xx or a network failure the lookup stops and backs off (Retry-After, else - 10 min). The lane waits at most 12 s from its start; a slower first lookup - leaves the cameras it had not reached on the flat prior until the next - 15-minute refresh, and keeps filling the cache meanwhile. + for 6 h, so a refresh only asks about new or moved cameras. After a + failing request (a 408, 429 or 5xx, a network error, a request over 10 s or + an unreadable body) the lookup stops and backs off (Retry-After, else + 30 min, longer than the refresh). The lane waits at most 12 s from its + start, and only on a cold cache; a slower first lookup leaves the cameras it + had not reached on the flat prior until the next 15-minute refresh, and + keeps filling the cache meanwhile. `CCTV_VEGVESEN_HEIGHTS=0` turns it off. Follow-up to #929 (trygvehhh, #945). diff --git a/server/providers/cctv/constants.js b/server/providers/cctv/constants.js index 553ff849f..8922e8938 100644 --- a/server/providers/cctv/constants.js +++ b/server/providers/cctv/constants.js @@ -347,22 +347,32 @@ export const KARTVERKET_HEIGHT_URL = * extra points from contour lines (`hoydekurver`, metres to ~20 m off) or not * at all; at 20 nearly every camera resolves from the 1 m terrain model. */ export const KARTVERKET_MAX_POINTS = 20; -/** Requests in flight at once. The whole country is ~36 requests of ~1-2 s, - * so a cold lookup takes ~6 s. */ +/** Requests in flight at once. The whole country is ~36 requests of + * 0.6-9.5 s each, so a cold lookup takes ~6-13 s. */ export const KARTVERKET_CONCURRENCY = 8; -/** Deadline for one complete lookup, shared by every request. */ +/** Deadline for one complete lookup, shared by every request. A lookup cut + * short by it is resumed on the next refresh, without a backoff. */ export const KARTVERKET_LOOKUP_TIMEOUT_MS = 30 * 1000; +/** One request that takes longer than this counts as a failing service. */ +export const KARTVERKET_REQUEST_TIMEOUT_MS = 10 * 1000; /** The Vegvesen lane's budget, measured from the start of the loader (catalog * fetch included), for waiting on heights. Under the 15 s per-provider fetch * timeout so this pack is never the catalog's slowest lane. A lookup still * running after it keeps filling the cache for the next refresh. */ export const KARTVERKET_CATALOG_WAIT_MS = 12 * 1000; +/** The lane only waits when more than this share of its cameras has no + * answer yet (a cold cache). A few new cameras or expired misses are looked + * up in the background instead of holding the refresh. */ +export const KARTVERKET_WAIT_MISSING_SHARE = 0.1; /** A point Kartverket has no height for (some ferry quays and bridges) is not * asked about again for this long. */ export const KARTVERKET_NULL_TTL_MS = 6 * 60 * 60 * 1000; -/** After a 429, a 5xx or a network failure, no lookup starts for this long, - * or for the Retry-After the service sent, bounded to [1 min, 1 h]. */ -export const KARTVERKET_FAILURE_BACKOFF_MS = 10 * 60 * 1000; +/** After a failing request (408, 429, 5xx, a network error, a request + * timeout or an unreadable body), no lookup starts for this long, or for the + * Retry-After the service sent, bounded to [1 min, 1 h]. Longer than the + * 15-minute catalog refresh (CCTV_SOURCE_CACHE_MS), so a failing service + * costs one attempt per backoff, not one per refresh. */ +export const KARTVERKET_FAILURE_BACKOFF_MS = 30 * 60 * 1000; export const KARTVERKET_MAX_BACKOFF_MS = 60 * 60 * 1000; export const KARTVERKET_MAX_RESPONSE_BYTES = 256 * 1024; /** Plausible Norwegian terrain: below sea level only at the shore, and no road diff --git a/server/providers/cctv/sources.js b/server/providers/cctv/sources.js index 38888c7db..4964f2aa0 100644 --- a/server/providers/cctv/sources.js +++ b/server/providers/cctv/sources.js @@ -99,6 +99,8 @@ import { KARTVERKET_MAX_POINTS, KARTVERKET_CONCURRENCY, KARTVERKET_LOOKUP_TIMEOUT_MS, + KARTVERKET_REQUEST_TIMEOUT_MS, + KARTVERKET_WAIT_MISSING_SHARE, KARTVERKET_CATALOG_WAIT_MS, KARTVERKET_NULL_TTL_MS, KARTVERKET_FAILURE_BACKOFF_MS, @@ -2506,12 +2508,20 @@ const KARTVERKET_CACHE_LIMIT = 5000; let kartverketPending = null; /** No lookup starts before this time (epoch ms) after a transport failure. */ let kartverketBackoffUntil = 0; +/** Bumped by clearKartverketHeightCache, so a lookup started before a clear + * can no longer write to the cache, the backoff or the pending slot. */ +let kartverketGeneration = 0; /** Cache key for one position, at the ~0.1 m precision the request sends. */ const kartverketKey = (lon, lat) => `${lon.toFixed(6)},${lat.toFixed(6)}`; +/** Statuses that say the service, not the request, is failing. */ +const isKartverketTransportStatus = (status) => + status === 408 || status === 429 || status >= 500; + /** Forget cached heights, the backoff and any lookup in flight. Tests only. */ export function clearKartverketHeightCache() { + kartverketGeneration += 1; kartverketHeightCache.clear(); kartverketPending = null; kartverketBackoffUntil = 0; @@ -2541,26 +2551,36 @@ export function kartverketBackoffMs(retryAfter, now = Date.now()) { * matched back by the coordinates the API echoes, and only for points that * request asked about, then reported through `into.set(key, z)` as its * request lands, so a caller that stops waiting keeps what has arrived. `z` is - * metres, or null when the API has no usable height there. + * metres, or null when the API has no usable height there. A point a 200 + * answer leaves out is reported as null too, so it is not asked again on + * every refresh. * * Over water `/punkt` returns the depth of the sea floor (`datakilde: * dybdekurver`, `terreng: Havflate`; −547 m under Nordhordlandsbrua). Either * marker counts as 0 m, the water surface. For the ~14 bridge cameras that is * below the deck, but far closer than the sea floor or a flat prior. * - * The first transport failure (a non-200 answer, a network error or the + * The first transport failure (a 408, 429 or 5xx, a network error, a request + * past KARTVERKET_REQUEST_TIMEOUT_MS, an unreadable or oversized body, or the * shared deadline) stops the lookup: no further requests start, and the * failure is returned so the caller can back off. Points in failed requests - * are not reported, so they are asked again after the backoff. + * are not reported, so they are asked again later. Any other 4xx rejects that + * request's points, not the service: they are reported as null and the lookup + * carries on. * * @param {Array<{lon:number, lat:number}>} points - * @param {{into?: {set: Function}, timeoutMs?: number}} [options] - * @returns {Promise<{heights: Map, failure: ?{status: ?number, retryAfter: ?string}}>} + * @param {{into?: {set: Function}, timeoutMs?: number, requestTimeoutMs?: number}} [options] + * @returns {Promise<{heights: Map, failure: ?{status: ?number, retryAfter: ?string, deadline: boolean}}>} * `heights` is `into` when it is a Map, else a Map of what was reported. + * `failure.deadline` is true when only the shared deadline ran out. */ export async function fetchKartverketHeights( points, - { into = new Map(), timeoutMs = KARTVERKET_LOOKUP_TIMEOUT_MS } = {}, + { + into = new Map(), + timeoutMs = KARTVERKET_LOOKUP_TIMEOUT_MS, + requestTimeoutMs = KARTVERKET_REQUEST_TIMEOUT_MS, + } = {}, ) { const heights = into instanceof Map ? into : new Map(); const report = (key, z) => { @@ -2580,7 +2600,7 @@ export async function fetchKartverketHeights( for (let i = 0; i < unique.length; i += KARTVERKET_MAX_POINTS) { batches.push(unique.slice(i, i + KARTVERKET_MAX_POINTS)); } - const signal = AbortSignal.timeout(timeoutMs); + const deadline = AbortSignal.timeout(timeoutMs); const fetchBatch = async (batch) => { const asked = new Set(batch.map((p) => kartverketKey(p.lon, p.lat))); const url = new URL(KARTVERKET_HEIGHT_URL); @@ -2597,7 +2617,10 @@ export async function fetchKartverketHeights( 'User-Agent': KARTVERKET_USER_AGENT, }, redirect: 'error', - signal, + signal: AbortSignal.any([ + deadline, + AbortSignal.timeout(requestTimeoutMs), + ]), }); if (!resp.ok) { try { @@ -2605,6 +2628,13 @@ export async function fetchKartverketHeights( } catch { /* no-op */ } + if (!isKartverketTransportStatus(resp.status)) { + console.warn( + `[CCTV] Kartverket rejected ${asked.size} points: HTTP ${resp.status}`, + ); + for (const key of asked) report(key, null); + return; + } const error = new Error(`HTTP ${resp.status}`); error.status = resp.status; error.retryAfter = resp.headers.get('retry-after'); @@ -2643,6 +2673,8 @@ export async function fetchKartverketHeights( z <= KARTVERKET_HEIGHT_RANGE_M.max; report(key, usable ? z : null); } + // A 200 that leaves a point out has no height for it either. + for (const key of asked) report(key, null); }; let next = 0; const worker = async () => { @@ -2655,6 +2687,7 @@ export async function fetchKartverketHeights( failure = { status: Number.isFinite(error?.status) ? error.status : null, retryAfter: error?.retryAfter ?? null, + deadline: deadline.aborted, }; console.warn( '[CCTV] Kartverket height lookup failed:', @@ -2675,23 +2708,35 @@ export async function fetchKartverketHeights( /** * Give each Vegvesen camera its own terrain height as the ground prior, in - * place. Waits at most `waitMs` for positions it has no answer for; a lookup - * still running then keeps filling the cache for the next catalog refresh. - * Cameras without a height keep the flat prior. After a transport failure no - * lookup starts until the backoff ends, so a failing Kartverket costs one - * attempt, not one per refresh. Never throws. CCTV_VEGVESEN_HEIGHTS=0 turns it - * off. + * place. Waits at most `waitMs` for positions it has no answer for, and only + * when more than KARTVERKET_WAIT_MISSING_SHARE of the cameras are missing + * (a cold cache); a few new cameras or expired misses are looked up in the + * background. A lookup still running when the wait ends keeps filling the + * cache for the next catalog refresh. Cameras without a height keep the flat + * prior. After a transport failure no lookup starts until the backoff ends, + * which outlasts the 15-minute catalog refresh, so a failing Kartverket costs + * one attempt per backoff, not one per refresh. A lookup cut short only by its + * shared deadline is resumed on the next refresh without a backoff. Cameras + * without finite coordinates are skipped. Never throws. + * CCTV_VEGVESEN_HEIGHTS=0 turns it off. * * @param {Array} cameras - Sources from vegvesenCameraToSource(). - * @param {{waitMs?: number, now?: () => number}} [options] + * @param {{waitMs?: number, now?: () => number, timeoutMs?: number}} [options] + * `timeoutMs` is the lookup's shared deadline. * @returns {Promise} How many cameras got a terrain height. */ export async function applyVegvesenGroundHeights( cameras, - { waitMs = KARTVERKET_CATALOG_WAIT_MS, now = Date.now } = {}, + { + waitMs = KARTVERKET_CATALOG_WAIT_MS, + now = Date.now, + timeoutMs = KARTVERKET_LOOKUP_TIMEOUT_MS, + } = {}, ) { if (String(process.env.CCTV_VEGVESEN_HEIGHTS || '1').trim() === '0') return 0; - const list = Array.isArray(cameras) ? cameras : []; + const list = (Array.isArray(cameras) ? cameras : []).filter( + (c) => Number.isFinite(c?.lon) && Number.isFinite(c?.lat), + ); const startedAt = now(); const missing = list.filter((c) => { const entry = kartverketHeightCache.get(kartverketKey(c.lon, c.lat)); @@ -2705,32 +2750,40 @@ export async function applyVegvesenGroundHeights( if (kartverketHeightCache.size + missing.length > KARTVERKET_CACHE_LIMIT) { kartverketHeightCache.clear(); } + const generation = kartverketGeneration; + const current = () => generation === kartverketGeneration; const sink = { - set: (key, z) => + set: (key, z) => { + if (!current()) return; kartverketHeightCache.set(key, { z, expires: z === null ? now() + KARTVERKET_NULL_TTL_MS : Infinity, - }), + }); + }, }; - kartverketPending = fetchKartverketHeights(missing, { into: sink }) + const pending = fetchKartverketHeights(missing, { into: sink, timeoutMs }) .then(({ failure }) => { - if (failure) { + if (failure && !failure.deadline && current()) { kartverketBackoffUntil = now() + kartverketBackoffMs(failure.retryAfter, now()); } }) .catch((error) => { - kartverketBackoffUntil = now() + KARTVERKET_FAILURE_BACKOFF_MS; + if (current()) { + kartverketBackoffUntil = now() + KARTVERKET_FAILURE_BACKOFF_MS; + } console.warn( '[CCTV] Kartverket height lookup error:', error?.message || error, ); }) .finally(() => { - kartverketPending = null; + if (kartverketPending === pending) kartverketPending = null; }); + kartverketPending = pending; } - if (missing.length && kartverketPending && waitMs > 0) { + const cold = missing.length > list.length * KARTVERKET_WAIT_MISSING_SHARE; + if (cold && kartverketPending && waitMs > 0) { let timer; await Promise.race([ kartverketPending, diff --git a/src/data/cctvVegvesen.test.mjs b/src/data/cctvVegvesen.test.mjs index 47ae732d1..3025f0d91 100644 --- a/src/data/cctvVegvesen.test.mjs +++ b/src/data/cctvVegvesen.test.mjs @@ -12,11 +12,14 @@ import { import { DEFAULT_VEGVESEN_CCTV_URL, KARTVERKET_HEIGHT_URL, + KARTVERKET_CATALOG_WAIT_MS, KARTVERKET_CONCURRENCY, KARTVERKET_FAILURE_BACKOFF_MS, KARTVERKET_MAX_POINTS, + KARTVERKET_MAX_RESPONSE_BYTES, KARTVERKET_NULL_TTL_MS, KARTVERKET_USER_AGENT, + CCTV_SOURCE_CACHE_MS, VEGVESEN_IMAGE_ORIGIN, } from '../../server/providers/cctv/constants.js'; import { createCctvCatalog } from '../../server/providers/cctv/catalog.js'; @@ -141,9 +144,9 @@ test('the loader dedupes, and failures degrade to an empty pack', async (t) => { const cameras = await loadVegvesenSourcesFromOpenData(); // The catalog first; the rest is the Kartverket height lookup. assert.equal(requested[0], DEFAULT_VEGVESEN_CCTV_URL); - assert.ok( - requested.slice(1).every((url) => url.startsWith(KARTVERKET_HEIGHT_URL)), - ); + // Two unique positions: one Kartverket request. + assert.equal(requested.length, 2); + assert.ok(requested[1].startsWith(KARTVERKET_HEIGHT_URL)); // Nearest-to-anchor first: Oslo leads. assert.deepEqual( cameras.map((c) => c.id), @@ -258,6 +261,8 @@ test('Kartverket heights come in batches of 20, matched by echoed coordinates', requested.push(String(url)); assert.ok(String(url).startsWith(KARTVERKET_HEIGHT_URL)); assert.equal(init.headers['User-Agent'], KARTVERKET_USER_AGENT); + // Redirects are refused, so a request can never leave the fixed host. + assert.equal(init.redirect, 'error'); // Reply in reverse order: matching must not depend on list position. const response = kartverketResponse(url, (x) => x > 7.04 ? null : Math.round((x - 7) * 10000), @@ -402,26 +407,138 @@ test('a transport failure stops the lookup: no retries, no new requests', async // after it starts, and no point is resent. assert.equal(calls, KARTVERKET_CONCURRENCY); assert.equal(heights.size, 0); - assert.deepEqual(failure, { status: 429, retryAfter: '120' }); + assert.deepEqual(failure, { + status: 429, + retryAfter: '120', + deadline: false, + }); }); +/** A fetch that never answers, and rejects only when its signal aborts. */ +const hangingFetch = (url, init) => { + assert.ok(init.signal instanceof AbortSignal); + return new Promise((_, reject) => { + init.signal.addEventListener('abort', () => reject(init.signal.reason)); + }); +}; + test('the lookup deadline bounds a hanging service', async (t) => { withHeightsEnv(t); - t.mock.method( - globalThis, - 'fetch', - (url, init) => - new Promise((_, reject) => { - init.signal.addEventListener('abort', () => reject(init.signal.reason)); - }), - ); + t.mock.method(globalThis, 'fetch', hangingFetch); const started = Date.now(); const { heights, failure } = await fetchKartverketHeights([cameraAt(0)], { timeoutMs: 30, }); - assert.ok(Date.now() - started < 2000); + const elapsed = Date.now() - started; + assert.ok(elapsed >= 25 && elapsed < 2000, `elapsed ${elapsed} ms`); assert.equal(heights.size, 0); - assert.equal(failure.status, null); + assert.deepEqual(failure, { status: null, retryAfter: null, deadline: true }); +}); + +test('one hanging request times out on its own, as a failing service', async (t) => { + withHeightsEnv(t); + t.mock.method(globalThis, 'fetch', hangingFetch); + const { failure } = await fetchKartverketHeights([cameraAt(0)], { + timeoutMs: 60_000, + requestTimeoutMs: 20, + }); + assert.deepEqual(failure, { + status: null, + retryAfter: null, + deadline: false, + }); +}); + +test('the deadline is shared: later requests do not get a fresh one', async (t) => { + withHeightsEnv(t); + let calls = 0; + t.mock.method(globalThis, 'fetch', (url, init) => { + calls += 1; + return new Promise((resolve, reject) => { + const timer = setTimeout( + () => resolve(kartverketResponse(url, () => 1)), + 40, + ); + init.signal.addEventListener('abort', () => { + clearTimeout(timer); + reject(init.signal.reason); + }); + }); + }); + // Five waves of 40 ms against a 100 ms deadline for the whole lookup. + const { heights, failure } = await fetchKartverketHeights( + Array.from({ length: 5 * KARTVERKET_CONCURRENCY * 20 }, (_, i) => + cameraAt(i), + ), + { timeoutMs: 100 }, + ); + assert.equal(failure?.deadline, true); + assert.ok(heights.size < 5 * KARTVERKET_CONCURRENCY * 20); + assert.ok(calls < 5 * KARTVERKET_CONCURRENCY, `calls ${calls}`); +}); + +test('a 503, an unreadable or an oversized body stops the lookup', async (t) => { + withHeightsEnv(t); + const answers = { + 503: () => new Response('down', { status: 503 }), + 'not json': () => + new Response('maintenance', { + headers: { 'Content-Type': 'text/html' }, + }), + oversized: () => + Response.json({ + punkter: [], + pad: 'x'.repeat(KARTVERKET_MAX_RESPONSE_BYTES + 1), + }), + }; + for (const [label, answer] of Object.entries(answers)) { + clearKartverketHeightCache(); + let calls = 0; + t.mock.method(globalThis, 'fetch', async () => { + calls += 1; + return answer(); + }); + const { heights, failure } = await fetchKartverketHeights( + Array.from({ length: 400 }, (_, i) => cameraAt(i)), + ); + assert.equal(calls, KARTVERKET_CONCURRENCY, label); + assert.equal(heights.size, 0, label); + assert.equal(failure?.deadline, false, label); + assert.equal(failure?.status, label === '503' ? 503 : null, label); + } +}); + +test('a 4xx rejects that request only: its points get no height, the rest go on', async (t) => { + withHeightsEnv(t); + t.mock.method(globalThis, 'fetch', async (url) => + sentPoints(url)[0][0] === 7 + ? new Response('bad point', { status: 400 }) + : kartverketResponse(url, () => 42), + ); + const { heights, failure } = await fetchKartverketHeights( + Array.from({ length: 40 }, (_, i) => cameraAt(i)), + ); + assert.equal(failure, null); + assert.equal(heights.size, 40); + assert.equal(heights.get('7.000000,60.000000'), null); + assert.equal(heights.get('7.020000,60.020000'), 42); +}); + +test('a point a 200 answer leaves out is cached as no height', async (t) => { + withHeightsEnv(t); + const fetchMock = t.mock.method(globalThis, 'fetch', async (url) => { + const points = sentPoints(url).slice(1); // drop the first point + return Response.json({ + koordsys: 4258, + punkter: points.map(([x, y]) => ({ x, y, z: 7 })), + }); + }); + const cameras = [cameraAt(0), cameraAt(1)]; + assert.equal(await applyVegvesenGroundHeights(cameras), 1); + assert.equal(cameras[0].groundElevationM, 150); + // The next refresh asks nothing: the left-out point is a cached miss. + assert.equal(await applyVegvesenGroundHeights([cameraAt(0), cameraAt(1)]), 1); + assert.equal(fetchMock.mock.callCount(), 1); }); test('Retry-After sets the backoff, bounded to [1 min, 1 h]', () => { @@ -478,8 +595,12 @@ test('after a failure no lookup starts until the backoff ends', async (t) => { assert.equal(cameras[0].groundElevationM, 150); assert.equal(fetchMock.mock.callCount(), 1); - // Refreshes inside the backoff cost nothing. - clock += KARTVERKET_FAILURE_BACKOFF_MS - 1; + // The default backoff outlasts the catalog refresh, so the next refresh + // and every one inside the backoff cost nothing. + assert.ok(KARTVERKET_FAILURE_BACKOFF_MS > CCTV_SOURCE_CACHE_MS); + clock += CCTV_SOURCE_CACHE_MS; + assert.equal(await applyVegvesenGroundHeights([cameraAt(0)], { now }), 0); + clock += KARTVERKET_FAILURE_BACKOFF_MS - CCTV_SOURCE_CACHE_MS - 1; assert.equal(await applyVegvesenGroundHeights([cameraAt(0)], { now }), 0); assert.equal(fetchMock.mock.callCount(), 1); @@ -494,6 +615,143 @@ test('after a failure no lookup starts until the backoff ends', async (t) => { assert.equal(fetchMock.mock.callCount(), 2); }); +test('a 429 backs off for its Retry-After, not the default', async (t) => { + withHeightsEnv(t); + let clock = 1_000_000; + const now = () => clock; + const retryAfterS = 50 * 60; + const fetchMock = t.mock.method( + globalThis, + 'fetch', + async () => + new Response('busy', { + status: 429, + headers: { 'Retry-After': String(retryAfterS) }, + }), + ); + await applyVegvesenGroundHeights([cameraAt(0)], { now }); + assert.equal(fetchMock.mock.callCount(), 1); + clock += KARTVERKET_FAILURE_BACKOFF_MS + 1; + await applyVegvesenGroundHeights([cameraAt(0)], { now }); + assert.equal(fetchMock.mock.callCount(), 1); + clock += retryAfterS * 1000 - KARTVERKET_FAILURE_BACKOFF_MS; + await applyVegvesenGroundHeights([cameraAt(0)], { now }); + assert.equal(fetchMock.mock.callCount(), 2); +}); + +test('a lookup cut short by its deadline resumes on the next refresh', async (t) => { + withHeightsEnv(t); + let clock = 1_000_000; + const now = () => clock; + t.mock.method(globalThis, 'fetch', hangingFetch); + // The lookup runs out of time; that is not a failing service. + assert.equal( + await applyVegvesenGroundHeights([cameraAt(0)], { + now, + timeoutMs: 10, + waitMs: 1000, + }), + 0, + ); + const fetchMock = t.mock.method(globalThis, 'fetch', async (url) => + kartverketResponse(url, () => 64), + ); + clock += CCTV_SOURCE_CACHE_MS; + const cameras = [cameraAt(0)]; + assert.equal(await applyVegvesenGroundHeights(cameras, { now }), 1); + assert.equal(cameras[0].groundElevationM, 64); + assert.equal(fetchMock.mock.callCount(), 1); +}); + +test('overlapping refreshes share one lookup', async (t) => { + withHeightsEnv(t); + let release; + const gate = new Promise((resolve) => { + release = resolve; + }); + const fetchMock = t.mock.method(globalThis, 'fetch', async (url) => { + await gate; + return kartverketResponse(url, () => 3); + }); + await Promise.all([ + applyVegvesenGroundHeights([cameraAt(0)], { waitMs: 5 }), + applyVegvesenGroundHeights([cameraAt(0)], { waitMs: 5 }), + ]); + assert.equal(fetchMock.mock.callCount(), 1); + release(); +}); + +test('a warm cache does not wait for a few new cameras', async (t) => { + withHeightsEnv(t); + const cameras = Array.from({ length: 20 }, (_, i) => cameraAt(i)); + t.mock.method(globalThis, 'fetch', async (url) => + kartverketResponse(url, () => 500), + ); + assert.equal(await applyVegvesenGroundHeights(cameras), 20); + let release; + const gate = new Promise((resolve) => { + release = resolve; + }); + const fetchMock = t.mock.method(globalThis, 'fetch', async (url) => { + await gate; + return kartverketResponse(url, () => 600); + }); + const refresh = [...cameras, cameraAt(20)]; + const started = Date.now(); + assert.equal( + await applyVegvesenGroundHeights(refresh, { waitMs: 5_000 }), + 20, + ); + assert.ok(Date.now() - started < 1000); + assert.equal(fetchMock.mock.callCount(), 1); + // The background lookup fills the cache for the next refresh. + release(); + await new Promise((resolve) => setTimeout(resolve, 20)); + const next = [...cameras, cameraAt(20)]; + assert.equal(await applyVegvesenGroundHeights(next), 21); + assert.equal(next[20].groundElevationM, 600); +}); + +test('cameras without numeric coordinates are skipped, never thrown on', async (t) => { + withHeightsEnv(t); + t.mock.method(globalThis, 'fetch', async (url) => + kartverketResponse(url, () => 9), + ); + const cameras = [ + { id: 'no-coordinates' }, + null, + { id: 'strings', lat: '60', lon: '7' }, + cameraAt(0), + ]; + assert.equal(await applyVegvesenGroundHeights(cameras), 1); + assert.equal(cameras[3].groundElevationM, 9); +}); + +test('a lookup started before a cache clear cannot write after it', async (t) => { + withHeightsEnv(t); + let release; + const gate = new Promise((resolve) => { + release = resolve; + }); + t.mock.method(globalThis, 'fetch', async (url) => { + await gate; + return kartverketResponse(url, () => 111); + }); + await applyVegvesenGroundHeights([cameraAt(0)], { waitMs: 5 }); + clearKartverketHeightCache(); + t.mock.method(globalThis, 'fetch', async (url) => + kartverketResponse(url, () => 222), + ); + const fresh = [cameraAt(0)]; + assert.equal(await applyVegvesenGroundHeights(fresh), 1); + assert.equal(fresh[0].groundElevationM, 222); + release(); + await new Promise((resolve) => setTimeout(resolve, 20)); + const later = [cameraAt(0)]; + await applyVegvesenGroundHeights(later, { waitMs: 0 }); + assert.equal(later[0].groundElevationM, 222); +}); + test('CCTV_VEGVESEN_HEIGHTS=0 skips the lookup', async (t) => { withHeightsEnv(t); const fetchMock = t.mock.method(globalThis, 'fetch', async (url) => @@ -541,3 +799,24 @@ test('the loader applies Kartverket heights to the cameras it returns', async (t [['no-vegvesen-3000063_1', 987.6]], ); }); + +test('the loader budgets the height wait from its own start', async (t) => { + withHeightsEnv(t); + const realNow = Date.now.bind(Date); + let offset = 0; + t.mock.method(Date, 'now', () => realNow() + offset); + t.mock.method(globalThis, 'fetch', (url, init) => { + if (String(url).startsWith(KARTVERKET_HEIGHT_URL)) { + return hangingFetch(url, init); + } + // The catalog fetch uses up all but 10 ms of the lane's budget. + offset += KARTVERKET_CATALOG_WAIT_MS - 10; + return Promise.resolve( + Response.json({ type: 'FeatureCollection', features: [feature()] }), + ); + }); + const started = performance.now(); + const cameras = await loadVegvesenSourcesFromOpenData(); + assert.ok(performance.now() - started < 1000); + assert.equal(cameras[0].groundElevationM, 150); +}); diff --git a/src/data/dataCredits.js b/src/data/dataCredits.js index f1aea1b74..b01492d9b 100644 --- a/src/data/dataCredits.js +++ b/src/data/dataCredits.js @@ -295,7 +295,8 @@ export const DATA_CREDITS = [ 'Norwegian licence for Open Government data (NLOD) ' + 'distributed by Statens vegvesen · camera ground heights © ' + 'Kartverket ' + - '(CC BY 4.0)', + '(CC BY 4.0); ' + + 'modified: heights over water set to 0 m and rounded to 0.1 m', }, { key: 'nyc-cctv', From 97630a598a544c647efa49dcbae8ca333f449c20 Mon Sep 17 00:00:00 2001 From: Said Turksever Date: Mon, 19 Oct 2026 23:15:43 +0100 Subject: [PATCH 8/9] fix(cctv): release Tarktee sibling on fetch rejection --- docs/CURRENT-STATE.md | 5 ++++ server/providers/cctv/sources.js | 10 +++++++- src/data/cctvEstonia.test.mjs | 40 ++++++++++++++++++++++++++++++++ 3 files changed, 54 insertions(+), 1 deletion(-) diff --git a/docs/CURRENT-STATE.md b/docs/CURRENT-STATE.md index 60937dd05..4b68e7030 100644 --- a/docs/CURRENT-STATE.md +++ b/docs/CURRENT-STATE.md @@ -4457,6 +4457,11 @@ silently demoting every later lookup for the session. empty keeps the old catalog and is retried after another 15 minutes, not on the next request. Estonia's Tarktee DATEX feeds, which answer in 2 to 16.6 s, get 30 s (`TARKTEE_FETCH_TIMEOUT_MS`) where other packs get 15 s. + Both Tarktee XML catalogs refuse redirects and have a 4 MiB per-document + limit, enforced against declared length and streamed bytes. A rejected fetch, + unsuccessful response, or body-read failure releases both available response + bodies; read failures also cancel an active sibling reader. Cameras sharing + an image number are deduplicated before the Tarktee pack cap is applied. - **CCTV v3 UX — viewshed + calibration gizmo** (built 2026-07-05 and field validated 2026-07-21): the COVERAGE toggle is a tri-state cycle `OFF → ON → VIEWSHED`; viewshed mode renders each visible camera's frustum diff --git a/server/providers/cctv/sources.js b/server/providers/cctv/sources.js index 7ccbf3aad..1e13df302 100644 --- a/server/providers/cctv/sources.js +++ b/server/providers/cctv/sources.js @@ -1200,7 +1200,7 @@ export function parseTarkteeDatexImages(xml) { */ export async function loadTarkteeSourcesFromDatex() { try { - const [locResp, imgResp] = await Promise.all([ + const responses = await Promise.allSettled([ fetch(TARKTEE_LOCATIONS_URL, { headers: { Accept: 'application/xml,text/xml,*/*' }, redirect: 'manual', @@ -1212,6 +1212,9 @@ export async function loadTarkteeSourcesFromDatex() { signal: AbortSignal.timeout(TARKTEE_FETCH_TIMEOUT_MS), }), ]); + const [locResp, imgResp] = responses.map((result) => + result.status === 'fulfilled' ? result.value : undefined, + ); // Both responses are fetched in parallel; if either is rejected or overflows // its byte cap, both bodies must be released so neither transport leaks. const discardBoth = async () => { @@ -1221,6 +1224,11 @@ export async function loadTarkteeSourcesFromDatex() { ]); return []; }; + const rejected = responses.find((result) => result.status === 'rejected'); + if (rejected) { + await discardBoth(); + throw rejected.reason; + } if ( (locResp.status >= 300 && locResp.status < 400) || (imgResp.status >= 300 && imgResp.status < 400) diff --git a/src/data/cctvEstonia.test.mjs b/src/data/cctvEstonia.test.mjs index 4c2a0f686..670432315 100644 --- a/src/data/cctvEstonia.test.mjs +++ b/src/data/cctvEstonia.test.mjs @@ -168,6 +168,46 @@ test('Tarktee loader refuses a redirect and cancels both response bodies', async assert.equal(cancelled.images, true, 'sibling 200 body is also cancelled'); }); +test('Tarktee loader cancels fulfilled bodies when either fetch rejects', async (t) => { + for (const failedUrl of [TARKTEE_LOCATIONS_URL, TARKTEE_IMAGES_URL]) { + for (const delayedSibling of [false, true]) { + await t.test( + `${failedUrl}, delayed sibling: ${delayedSibling}`, + async (t) => { + quiet(t); + let cancelled = false; + const response = new Response( + new ReadableStream({ + start(controller) { + controller.enqueue(new TextEncoder().encode('')); + }, + cancel() { + cancelled = true; + }, + }), + ); + t.mock.method(globalThis, 'fetch', async (url) => { + if (String(url) === failedUrl) throw new TypeError('fetch failed'); + if (delayedSibling) await new Promise(setImmediate); + return response; + }); + + try { + assert.deepEqual(await loadTarkteeSourcesFromDatex(), []); + assert.equal( + cancelled, + true, + 'fulfilled sibling body is cancelled', + ); + } finally { + await response.body.cancel(); + } + }, + ); + } + } +}); + test('Tarktee loader cancels the sibling body when one feed fails or overflows the byte cap', async (t) => { quiet(t); let siblingCancelled = false; From f0e0ea42ce6025d18af9e8eb195f93444d368d12 Mon Sep 17 00:00:00 2001 From: Said Turksever Date: Mon, 19 Oct 2026 23:31:52 +0100 Subject: [PATCH 9/9] test(cctv): verify DEM placement in the keyless globe gate --- scripts/qa-cctv-v2.mjs | 101 +++++++++++++++++++++++++++++++++-------- 1 file changed, 83 insertions(+), 18 deletions(-) diff --git a/scripts/qa-cctv-v2.mjs b/scripts/qa-cctv-v2.mjs index 2c121e0d6..88e0a5a2c 100644 --- a/scripts/qa-cctv-v2.mjs +++ b/scripts/qa-cctv-v2.mjs @@ -11,8 +11,10 @@ * exactly once per camera ACTIVATION (the §9.1 obstruction probe) and * never again in steady state; shared mesh-floor sampling fires at most * once per coarse cell during the staggered geometry-load drain, and - * actually COMPLETES once tiles load (>=1 real sample — the - * ceiling alone would pass trivially at Δ=0 if the snap never finished), + * actually COMPLETES once tiles load in photoreal mode (>=1 real sample + * — the ceiling alone would pass trivially at Δ=0 if the snap never + * finished). Globe mode instead proves the rendered mount uses a real + * terrain height with zero mesh samples. Sampling * stays flat while idle, and stays flat for non-positional pose edits. * 2. Geometry contract: the 5-polyline frustum wireframe + far-cap plane * exist for the active camera, the plane's corners coincide with the @@ -658,24 +660,87 @@ async function main() { // (existence, not just ceiling). The wait below tolerates the update-tick // latency of the completion pass (update interval is 10s). // ----------------------------------------------------------------------- - console.log('Waiting for tiles + the one-shot completion pass...'); - const tilesSeenLoaded = await waitForTilesLoaded(page, 45000); - let sampleFloorOk = false; - if (tilesSeenLoaded) { - sampleFloorOk = await waitForPredicate( - page, - (base) => window.__qaCounters.sampleHeight - base >= 1, - { timeoutMs: 30000, args: [c0.sampleHeight] }, + const globeMode = await page.evaluate( + () => window.__godsEyeView.viewer.scene.globe?.show !== false, + ); + if (globeMode) { + // Globe stacks intentionally perform no scene.sampleHeight query. + // Require independently fetched DEM heights and the actual rendered + // mount, so a catalog/geoid fallback cannot turn zero samples into a pass. + console.log( + 'Checking terrain-globe completion against live DEM heights...', + ); + const terrain = await page.evaluate(async () => { + const camera = window.__godsEyeView.dataManager.layers + .get('cctv') + .module.getUIState().activeCamera; + const exact = `${camera.lon.toFixed(5)},${camera.lat.toFixed(5)}`; + const cell = `${camera.lon.toFixed(3)},${camera.lat.toFixed(3)}`; + const response = await fetch( + `/api/terrain/heights?points=${encodeURIComponent(`${exact};${cell}`)}`, + { signal: AbortSignal.timeout(30000) }, + ); + const body = await response.json(); + return { ok: response.ok, results: body.results }; + }); + const heights = terrain.results?.map((result) => result?.ellipsoid); + const terrainReady = + terrain.ok && heights?.length === 2 && heights.every(Number.isFinite); + const mountResolved = + terrainReady && + (await waitForPredicate( + page, + (expected, base) => { + const { viewer, dataManager } = window.__godsEyeView; + const camera = dataManager.layers + .get('cctv') + .module.getUIState().activeCamera; + const ray = viewer.entities.getById(`cctv-${camera.id}-ray-tl`); + const mount = ray?.polyline?.positions.getValue( + viewer.clock.currentTime, + )?.[0]; + const height = + mount && + viewer.scene.globe.ellipsoid.cartesianToCartographic(mount) + .height; + return ( + Number.isFinite(camera.groundPriorM) && + Math.abs(camera.groundPriorM - expected[0]) < 0.01 && + Number.isFinite(height) && + expected.some( + (ground) => + Math.abs(height - ground - camera.mountHeightM) < 0.05, + ) && + window.__qaCounters.sampleHeight === base + ); + }, + { timeoutMs: 30000, args: [heights, c0.sampleHeight] }, + )); + record( + 'terrain-globe completion uses a real DEM height at the rendered mount without mesh samples', + Boolean(mountResolved), + `terrain=${JSON.stringify(heights)}, mesh sample Δ=${(await readCounters()).sampleHeight - c0.sampleHeight}`, + ); + } else { + console.log('Waiting for tiles + the one-shot completion pass...'); + const tilesSeenLoaded = await waitForTilesLoaded(page, 45000); + let sampleFloorOk = false; + if (tilesSeenLoaded) { + sampleFloorOk = await waitForPredicate( + page, + (base) => window.__qaCounters.sampleHeight - base >= 1, + { timeoutMs: 30000, args: [c0.sampleHeight] }, + ); + } + const cFloor = await readCounters(); + record( + '>=1 real ground sample once tilesLoaded observed (one-shot snap completed)', + tilesSeenLoaded ? sampleFloorOk : null, + tilesSeenLoaded + ? `total real samples since enable=${cFloor.sampleHeight - c0.sampleHeight}` + : 'tilesLoaded never observed within 45s — floor not assessable this run', ); } - const cFloor = await readCounters(); - record( - '>=1 real ground sample once tilesLoaded observed (one-shot snap completed)', - tilesSeenLoaded ? sampleFloorOk : null, - tilesSeenLoaded - ? `total real samples since enable=${cFloor.sampleHeight - c0.sampleHeight}` - : 'tilesLoaded never observed within 45s — floor not assessable this run', - ); // Let the completion pass finish before the steady-state window: require // the sampleHeight counter quiet for 12s (> the 10s update interval, so