@basemaps/lambda-tiler 7.14.0 → 7.16.0

package/dist/node_modules/semver/functions/diff.js

@@ -27,20 +27,13 @@ const diff = (version1, version2) => {
       return 'major'
     }
 
-    // Otherwise it can be determined by checking the high version
-
-    if (highVersion.patch) {
-      // anything higher than a patch bump would result in the wrong version
+    // If the main part has no difference
+    if (lowVersion.compareMain(highVersion) === 0) {
+      if (lowVersion.minor && !lowVersion.patch) {
+        return 'minor'
+      }
       return 'patch'
     }
-
-    if (highVersion.minor) {
-      // anything higher than a minor bump would result in the wrong version
-      return 'minor'
-    }
-
-    // bumping major/minor/patch all have same result
-    return 'major'
   }
 
   // add the `pre` prefix if we are going to a prerelease version

package/dist/node_modules/semver/internal/re.js

@@ -10,6 +10,7 @@ exports = module.exports = {}
 const re = exports.re = []
 const safeRe = exports.safeRe = []
 const src = exports.src = []
+const safeSrc = exports.safeSrc = []
 const t = exports.t = {}
 let R = 0
 
@@ -42,6 +43,7 @@ const createToken = (name, value, isGlobal) => {
   debug(name, index, value)
   t[name] = index
   src[index] = value
+  safeSrc[index] = safe
   re[index] = new RegExp(value, isGlobal ? 'g' : undefined)
   safeRe[index] = new RegExp(safe, isGlobal ? 'g' : undefined)
 }

package/dist/node_modules/semver/package.json

@@ -1,20 +1,21 @@
 {
   "name": "semver",
-  "version": "7.6.3",
+  "version": "7.7.1",
   "description": "The semantic version parser used by npm.",
   "main": "index.js",
   "scripts": {
     "test": "tap",
     "snap": "tap",
-    "lint": "eslint \"**/*.{js,cjs,ts,mjs,jsx,tsx}\"",
+    "lint": "npm run eslint",
     "postlint": "template-oss-check",
-    "lintfix": "npm run lint -- --fix",
+    "lintfix": "npm run eslint -- --fix",
     "posttest": "npm run lint",
-    "template-oss-apply": "template-oss-apply --force"
+    "template-oss-apply": "template-oss-apply --force",
+    "eslint": "eslint \"**/*.{js,cjs,ts,mjs,jsx,tsx}\""
   },
   "devDependencies": {
-    "@npmcli/eslint-config": "^4.0.0",
-    "@npmcli/template-oss": "4.22.0",
+    "@npmcli/eslint-config": "^5.0.0",
+    "@npmcli/template-oss": "4.23.4",
     "benchmark": "^2.1.4",
     "tap": "^16.0.0"
   },
@@ -51,7 +52,7 @@
   "author": "GitHub Inc.",
   "templateOSS": {
     "//@npmcli/template-oss": "This file is partially managed by @npmcli/template-oss. Edits may be overwritten.",
-    "version": "4.22.0",
+    "version": "4.23.4",
     "engines": ">=10",
     "distPaths": [
       "classes/",

package/dist/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@basemaps/lambda-tiler",
-  "version": "7.14.0",
+  "version": "7.16.0",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@basemaps/lambda-tiler",
-      "version": "7.14.0",
+      "version": "7.16.0",
       "license": "MIT",
       "dependencies": {
         "lerc": "4.0.4",
@@ -539,9 +539,9 @@
       "license": "Apache-2.0"
     },
     "node_modules/semver": {
-      "version": "7.6.3",
-      "resolved": "https://registry.npmjs.org/semver/-/semver-7.6.3.tgz",
-      "integrity": "sha512-oVekP1cKtI+CTDvHWYFUcMtsK/00wmAEfyqKfNdARm8u1wNVhSgaX7A8d4UuIlUI5e84iEwOhs7ZPYRmzU9U6A==",
+      "version": "7.7.1",
+      "resolved": "https://registry.npmjs.org/semver/-/semver-7.7.1.tgz",
+      "integrity": "sha512-hlq8tAfn0m/61p4BVRcPzIGr6LKiMwo4VM6dGi6pt4qcRkmNzTcWq6eCEjEh+qXjkMDvPlOFFSGwQjoEa6gyMA==",
       "license": "ISC",
       "bin": {
         "semver": "bin/semver.js"

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@basemaps/lambda-tiler",
-  "version": "7.14.0",
+  "version": "7.16.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/linz/basemaps.git",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@basemaps/lambda-tiler",
-  "version": "7.14.0",
+  "version": "7.16.0",
   "repository": {
     "type": "git",
     "url": "https://github.com/linz/basemaps.git",
@@ -22,12 +22,12 @@
   "types": "./build/index.d.ts",
   "license": "MIT",
   "dependencies": {
-    "@basemaps/config": "^7.14.0",
-    "@basemaps/config-loader": "^7.14.0",
-    "@basemaps/geo": "^7.12.0",
-    "@basemaps/shared": "^7.14.0",
-    "@basemaps/tiler": "^7.12.0",
-    "@basemaps/tiler-sharp": "^7.14.0",
+    "@basemaps/config": "^7.16.0",
+    "@basemaps/config-loader": "^7.16.0",
+    "@basemaps/geo": "^7.16.0",
+    "@basemaps/shared": "^7.16.0",
+    "@basemaps/tiler": "^7.16.0",
+    "@basemaps/tiler-sharp": "^7.16.0",
     "@linzjs/geojson": "^7.10.0",
     "@linzjs/lambda": "^4.0.0",
     "@mapbox/vector-tile": "^2.0.3",
@@ -50,11 +50,11 @@
     "bundle": "./bundle.sh"
   },
   "devDependencies": {
-    "@basemaps/attribution": "^7.12.0",
+    "@basemaps/attribution": "^7.16.0",
     "@chunkd/fs": "^11.2.0",
     "@types/aws-lambda": "^8.10.75",
     "@types/pixelmatch": "^5.0.0",
     "pretty-json-log": "^1.0.0"
   },
-  "gitHead": "72f2a811d4851c6f50758617c9e65179d2b27bc8"
+  "gitHead": "25fa7659b84a40205ceacda8b2ca5f48e1dc6f7b"
 }

package/src/__tests__/config.data.ts

@@ -63,6 +63,24 @@ export const TileSetElevation: ConfigTileSetRaster = {
   ],
   outputs: [DefaultTerrainRgbOutput, DefaultColorRampOutput],
 };
+export const TileSetHillshadeElevation: ConfigTileSetRaster = {
+  id: 'ts_hillshade',
+  name: 'hillshade',
+  type: TileSetType.Raster,
+  description: 'hillshade__description',
+  title: 'hillshade Imagery',
+  category: 'hillshade',
+  layers: [
+    {
+      // TODO: create one band imagery to reference
+      2193: 'im_01FYWKAJ86W9P7RWM1VB62KD0H',
+      3857: 'im_01FYWKATAEK2ZTJQ2PX44Y0XNT',
+      title: 'New Zealand 8m Hillshade DEM (2012)',
+      name: 'new-zealand_hillshade_2012_dem_8m',
+    },
+  ],
+  outputs: [DefaultColorRampOutput],
+};
 
 export const Imagery2193: ConfigImagery = {
   id: 'im_01FYWKAJ86W9P7RWM1VB62KD0H',

package/src/routes/__tests__/fonts.test.ts

@@ -8,6 +8,7 @@ import { fsa, FsMemory } from '@chunkd/fs';
 import { Api, mockRequest, mockUrlRequest } from '../../__tests__/xyz.util.js';
 import { handler } from '../../index.js';
 import { CachedConfig } from '../../util/config.cache.js';
+import { ConfigLoader } from '../../util/config.loader.js';
 import { CoSources } from '../../util/source.cache.js';
 import { fontList } from '../fonts.js';
 
@@ -28,12 +29,15 @@ describe('/v1/fonts', () => {
     memory.files.clear();
   });
 
-  it('should return 404 if no font found', async () => {
+  it('should return 404 if no font found', async (t) => {
+    t.mock.method(ConfigLoader, 'getDefaultConfig', () => config);
     const res = await fontList(mockRequest('/v1/fonts.json'));
     assert.equal(res.status, 404);
   });
 
-  it('should return a list of fonts found', async () => {
+  it('should return a list of fonts found', async (t) => {
+    t.mock.method(ConfigLoader, 'getDefaultConfig', () => config);
+
     await fsa.write(
       new URL('memory://assets/fonts/fonts.json'),
       Buffer.from(JSON.stringify(['Roboto Black', 'Roboto Thin'])),
@@ -45,7 +49,9 @@ describe('/v1/fonts', () => {
     assert.equal(res._body?.toString(), JSON.stringify(['Roboto Black', 'Roboto Thin']));
   });
 
-  it('should get the correct font', async () => {
+  it('should get the correct font', async (t) => {
+    t.mock.method(ConfigLoader, 'getDefaultConfig', () => config);
+
     await fsa.write(new URL('memory://assets/fonts/Roboto Thin/0-255.pbf'), Buffer.from(''));
     const res255 = await handler.router.handle(mockRequest('/v1/fonts/Roboto Thin/0-255.pbf'));
     assert.equal(res255.status, 200);
@@ -58,7 +64,9 @@ describe('/v1/fonts', () => {
     assert.equal(res404.status, 404);
   });
 
-  it('should get the correct utf8 font', async () => {
+  it('should get the correct utf8 font', async (t) => {
+    t.mock.method(ConfigLoader, 'getDefaultConfig', () => config);
+
     await fsa.write(new URL('memory://assets/fonts/🦄 🌈/0-255.pbf'), Buffer.from(''));
     const res255 = await handler.router.handle(mockRequest('/v1/fonts/🦄 🌈/0-255.pbf'));
     assert.equal(res255.status, 200);
@@ -68,7 +76,9 @@ describe('/v1/fonts', () => {
     assert.equal(res255.header('cache-control'), 'public, max-age=604800, stale-while-revalidate=86400');
   });
 
-  it('should return 404 if no asset location set', async () => {
+  it('should return 404 if no asset location set', async (t) => {
+    t.mock.method(ConfigLoader, 'getDefaultConfig', () => config);
+
     config.assets = undefined;
     const res = await fontList(mockRequest('/v1/fonts.json'));
     assert.equal(res.status, 404);
@@ -76,11 +86,14 @@ describe('/v1/fonts', () => {
 
   it('should get the correct utf8 font with default assets', async () => {
     config.assets = undefined;
+    const configJson = config.toJson();
+    await fsa.write(new URL(`memory://config-${configJson.hash}.json`), JSON.stringify(configJson));
+
     config.objects.set('cb_latest', {
       id: 'cb_latest',
       name: 'latest',
-      path: 'latest',
-      hash: 'hash',
+      path: `memory://config-${configJson.hash}.json`,
+      hash: configJson.hash,
       assets: 'memory://new-location/',
     } as BaseConfig);
 
@@ -93,7 +106,9 @@ describe('/v1/fonts', () => {
     assert.equal(res255.header('cache-control'), 'public, max-age=604800, stale-while-revalidate=86400');
   });
 
-  it('should get the correct utf8 font with config assets', async () => {
+  it('should get the correct utf8 font with config assets', async (t) => {
+    t.mock.method(ConfigLoader, 'getDefaultConfig', () => config);
+
     const cfgBundle = new ConfigProviderMemory();
     cfgBundle.assets = 'memory://config/assets/';
     await fsa.write(new URL('memory://linz-basemaps/bar.json'), JSON.stringify(cfgBundle.toJson()));

package/src/routes/__tests__/xyz.test.ts

@@ -135,7 +135,10 @@ describe('/v1/tiles', () => {
 
     const elevation = FakeData.tileSetRaster('elevation');
 
-    elevation.outputs = [{ title: 'Terrain RGB', name: 'terrain-rgb' }];
+    elevation.outputs = [
+      { title: 'Terrain RGB', name: 'terrain-rgb' },
+      { title: 'Color Ramp', name: 'color-ramp' },
+    ];
     config.put(elevation);
 
     const request = mockRequest('/v1/tiles/elevation/3857/11/2022/1283.webp', 'get', Api.header);
@@ -145,6 +148,21 @@ describe('/v1/tiles', () => {
     assert.equal(res.status, 404, res.statusDescription);
   });
 
+  it('should default to the pipeline if only one pipeline is defined', async (t) => {
+    t.mock.method(ConfigLoader, 'getDefaultConfig', () => Promise.resolve(config));
+
+    const elevation = FakeData.tileSetRaster('elevation');
+
+    elevation.outputs = [{ title: 'Terrain RGB', name: 'terrain-rgb' }];
+    config.put(elevation);
+
+    const request = mockRequest('/v1/tiles/elevation/3857/11/2022/1283.webp', 'get', Api.header);
+
+    const res = await handler.router.handle(request);
+
+    assert.equal(res.status, 200, res.statusDescription);
+  });
+
   it('should generate a terrain-rgb 11/2022/1283 in webp', async (t) => {
     t.mock.method(ConfigLoader, 'getDefaultConfig', () => Promise.resolve(config));
 

package/src/util/config.loader.ts

@@ -18,13 +18,21 @@ const SafeProtocols = new Set([new URL('s3://foo').protocol, new URL('memory://f
 
 export class ConfigLoader {
   /** Exposed for testing */
-  static async getDefaultConfig(): Promise<BasemapsConfigProvider> {
+  static async getDefaultConfig(req?: LambdaHttpRequest): Promise<BasemapsConfigProvider> {
     const config = getDefaultConfig();
-    if (config.assets == null) {
-      const cb = await config.ConfigBundle.get(config.ConfigBundle.id('latest'));
-      if (cb) config.assets = cb.assets;
-    }
-    return config;
+
+    // Look up the latest config bundle out of dynamodb, then load the config from the provided path
+    const cb = await config.ConfigBundle.get('cb_latest');
+    if (cb == null) return config;
+
+    req?.timer.start('config:load');
+
+    return CachedConfig.get(fsa.toUrl(cb.path)).then((cfg) => {
+      req?.timer.end('config:load');
+      if (cfg == null) throw new LambdaHttpResponse(500, 'Unable to find latest configuration');
+      if (cfg.assets == null) cfg.assets = cb.assets;
+      return cfg;
+    });
   }
 
   /** Lookup the config path from a request and return a standardized location */

package/src/util/validate.ts

@@ -125,6 +125,9 @@ export const Validate = {
    * Defaults to standard image format output if no outputs are defined on the tileset
    */
   pipeline(tileSet: ConfigTileSetRaster, tileType: string, pipeline?: string | null): ConfigTileSetRasterOutput | null {
+    // If there is only one pipeline force the use of it
+    if (tileSet.outputs?.length === 1 && pipeline == null) pipeline = tileSet.outputs[0].name;
+
     if (pipeline != null && pipeline !== 'rgba') {
       if (tileSet.outputs == null) throw new LambdaHttpResponse(404, 'TileSet has no pipelines');
       const output = tileSet.outputs.find((f) => f.name === pipeline);
@@ -136,7 +139,8 @@ export const Validate = {
       }
       return output;
     }
-    // If the tileset has pipelines defined the user MUST specify which one
+
+    // If the tileset has multiple pipelines defined the user MUST specify which one
     if (tileSet.outputs) {
       throw new LambdaHttpResponse(404, 'TileSet needs pipeline: ' + tileSet.outputs.map((f) => f.name).join(', '));
     }