@masterportal/masterportalapi 2.36.0 → 2.38.0

package/CHANGELOG.md

@@ -3,6 +3,34 @@
 
  The [Semantic Versioning](https://semver.org/spec/v2.0.0.html) is used.
 
+## 2.38.0 - 2024-06-04
+
+### Added
+- Added support for Node LTS version ^20.12.2 and npm Version ^10.5.0.
+
+### Changed
+- The following packages have been updated:
+    - dependencies:
+        - ol: 9.1.0 to 9.2.4
+        - olcs: 2.19.3 to 2.20.0
+
+### Fixed
+- The mitigation for issue 666 in ol-cesium since version 2.37.0 is removed
+- Webgl: vectorTiles were not displayed.
+- WMS-Layer: special loadFunction is only used if layer's attribute `isSecured` is true.
+
+---
+
+## 2.37.0 - 2024-04-23
+
+### Fixed
+- Issue Masterportal #1114: `isSecured` parameter is considered for WMS load function
+- Issue Masterportal #1114: custom loading function for WMS is now applied on non-tiled WMS, too.
+  However, because of issue 666 in ol-cesium it does not work for 3D and is enabled for `isSecured: true` layers only
+- Image layers that cannot be converted by olcs are now properly handled
+
+---
+
 ## 2.36.0 - 2024-04-11
 
 ### Changed

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@masterportal/masterportalapi",
   "author": "Implementierungspartnerschaft Masterportal <info@masterportal.org> (https://www.masterportal.org)",
-  "version": "2.36.0",
+  "version": "2.38.0",
   "license": "MIT",
   "description": "Basic functions of the Masterportal as api",
   "repository": {
@@ -21,9 +21,9 @@
   "dependencies": {
     "core-js": "^3.33.1",
     "dayjs": "^1.11.10",
-    "ol": "9.1.0",
+    "ol": "9.2.4",
     "ol-mapbox-style": "12.2.1",
-    "olcs": "2.19.3",
+    "olcs": "2.20.0",
     "proj4": "^2.10.0",
     "xml2js": "^0.6.2"
   },
@@ -59,8 +59,8 @@
     "@cesium/engine": "^6.2.0"
   },
   "engines": {
-    "node": "^16.13.2 || ^18.12.0 || ^20.9.0",
-    "npm": "^8.1.2 || ^9.3.1 || ^10.1.0"
+    "node": "^16.13.2 || ^18.12.0 || ^20.12.2",
+    "npm": "^8.1.2 || ^9.3.1 || ^10.5.0"
   },
   "homepage": "https://bitbucket.org/geowerkstatt-hamburg/masterportalapi#readme",
   "directories": {

package/src/layer/terrain.js

@@ -26,7 +26,7 @@ async function createTerrainProvider (rawLayer) {
  * @param {OLCesium} map ol cesium map
  * @returns {void}
  */
-async function setVisible (value, rawLayer, map) {
+export async function setVisible (value, rawLayer, map) {
     if (map && typeof map.getCesiumScene === "function") {
         if (value) {
             map.getCesiumScene().terrainProvider = await createTerrainProvider(rawLayer);
@@ -64,5 +64,3 @@ export function get (key) {
     }
     return this.values[key];
 }
-
-module.exports.setVisible = setVisible;

package/src/layer/wms.js

@@ -76,11 +76,17 @@ export function makeParams (rawLayer) {
     return params;
 }
 
-async function defaultTileLoadFunction (imageTile, src) {
-    const response = await fetch(src),
-        data = await response.blob();
+/**
+ * Return a ol tile or image load function for the rawLayer. Fetches with credentials: 'include'.
+ * @returns {function} tile/image load function
+ */
+export function loadFunctionWithCredentials () {
+    return async (image, src) => {
+        const response = await fetch(src, {credentials: "include"}),
+            data = await response.blob();
 
-    imageTile.getImage().src = URL.createObjectURL(data);
+        image.getImage().src = URL.createObjectURL(data);
+    };
 }
 
 /**
@@ -93,6 +99,7 @@ async function defaultTileLoadFunction (imageTile, src) {
  * @param {string|number} [rawLayer.tilesize] - optional needed to create the tileGrid
  * @param {string|"anonymous"} [rawLayer.crossOrigin] - optional needed for CORS requests of image data, to enable canvas export
  * @param {string} [rawLayer.cqlFilter] - optional CQL filter to add to the request
+ * @param {boolean} [rawLayer.isSecured] - if the layer is secured by BasicAuth, images are loaded with credentials: 'include'.
  * @param {object} options - optional resolutions and origin to create the TileGrid
  * @param {Array} [options.resolutions] - optional resolutions to create the TileGrid, must be in descending order
  * @param {Array} [options.origin] - optional origin to create the TileGrid
@@ -111,7 +118,10 @@ export function createLayerSource (rawLayer, options) {
             serverType: rawLayer.serverType,
             projection: projection,
             attributions: rawLayer.olAttribution,
-            crossOrigin: rawLayer.crossOrigin
+            crossOrigin: rawLayer.crossOrigin,
+            ...rawLayer.isSecured ? {
+                imageLoadFunction: loadFunctionWithCredentials()
+            } : {}
         });
     }
     if (options && options.resolutions) {
@@ -129,7 +139,9 @@ export function createLayerSource (rawLayer, options) {
         projection: projection,
         attributions: rawLayer.olAttribution,
         crossOrigin: rawLayer.crossOrigin,
-        tileLoadFunction: defaultTileLoadFunction
+        ...rawLayer.isSecured ? {
+            tileLoadFunction: loadFunctionWithCredentials()
+        } : {}
     });
 }
 

package/src/maps/ol/olMap.js

@@ -148,7 +148,8 @@ export function createMap (config = defaults, {mapParams, callback, errorCallbac
     crs.registerProjections(config.namedProjections);
     setBackgroundImage(config);
     setGazetteerUrl(config.gazetteerUrl);
-    const selectedInteractions = config.mapInteractions?.interactionModes ? config.mapInteractions?.interactionModes : {dragPan: false, altShiftDragRotate: false, pinchRotate: false},
+
+    const selectedInteractions = Object.assign({}, {dragPan: false, altShiftDragRotate: false, pinchRotate: false}, config.mapInteractions?.interactionModes),
         map = new Map(Object.assign({
             target: config.target || defaults.target,
             interactions: olDefaultInteractions(selectedInteractions).extend([
@@ -157,7 +158,7 @@ export function createMap (config = defaults, {mapParams, callback, errorCallbac
                         if (event.originalEvent.shiftKey) {
                             return false;
                         }
-                        return (!event.originalEvent.pointerType || event.originalEvent.pointerType === "mouse") || (config.twoFingerPan && this.getPointerCount() === 2) || !config.twoFingerPan;
+                        return (!event.originalEvent.pointerType || event.originalEvent.pointerType === "mouse") || (config.mapInteractions?.interactionModes?.twoFingerPan && this.getPointerCount() === 2) || !config.mapInteractions?.interactionModes?.twoFingerPan;
                     }
                 })
             ]),

package/src/maps/olcs/3dUtils/wmsRasterSynchronizer.js

@@ -112,7 +112,8 @@ class WMSRasterSynchronizer extends AbstractSynchronizer {
             provider = this.createProviderForTileWMS(source, viewProj, olLayer);
         }
         else if (source instanceof ImageWMS) {
-            return [this.createImageryLayerForImageWMS(olLayer, viewProj)];
+            cesiumLayer = this.createImageryLayerForImageWMS(olLayer, viewProj);
+            return cesiumLayer ? [cesiumLayer] : null;
         }
         else if (source instanceof StaticImageSource) {
             provider = this.createProviderForStaticImageSource(source);

package/src/renderer/webgl.js

@@ -107,6 +107,7 @@ export function createVectorTileLayerRenderer () {
                         fragment: result.builder.getSymbolFragmentShader(),
                         vertex: result.builder.getSymbolVertexShader()
                     },
+                    builder: result.builder,
                     attributes: {
                         fillColor: {
                             size: 2,

package/test/layer/wms.test.js

@@ -102,6 +102,33 @@ describe("wms.js", function () {
         });
     });
 
+    describe("loadFunctionWithCredentials", function () {
+        beforeEach(() => {
+            const blob = new Blob();
+
+            global.fetch = jest.fn().mockImplementation(async (url, {credentials}) => {
+                if ((url === "SECURE_WMS" && credentials === "include") ||
+                    (url === "INSECURE_WMS" && (typeof credentials === "undefined" || credentials === "omit"))) {
+                    return {
+                        ok: true,
+                        status: 200,
+                        blob: () => Promise.resolve(blob)
+                    };
+                }
+                throw new Error("Bad request");
+            });
+            URL.createObjectURL = jest.fn().mockImplementation(value => value === blob ? "blob:BLOB" : null);
+        });
+
+        it("passes credentials if isSecured is set", async function () {
+            const result = wms.loadFunctionWithCredentials(),
+                input = {src: ""};
+
+            await result({getImage: () => input}, "SECURE_WMS");
+            expect(input.src).toEqual("blob:BLOB");
+        });
+    });
+
     describe("createLayerSource", function () {
         it("creates a TileWMS for multi tile requests", function () {
             const source = wms.createLayerSource({singleTile: false});