uuid layer metadata comparison endpoint

Author: honza2

CHANGELOG.md

@@ -25,6 +25,7 @@
 - [#1126](https://github.com/LayerManager/layman/issues/1126) Endpoint [Workspace Layers](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#workspace-layers) (with methods GET, POST, DELETE) was unified into endpoint [Layers](doc/rest.md#get-layers); `workspace` is now supplied via request query/body parameter instead of URL path.
 - [#1126](https://github.com/LayerManager/layman/issues/1126) Endpoint [Workspace Layer Chunk](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#workspace-layer-chunk) (with methods GET, POST) was removed and replaced with endpoint [Layer Chunk](doc/rest.md#layer-chunk) to use UUID-based URL `/rest/layers/{uuid}/chunk` instead of workspace&name-based URL.
 - [#1126](https://github.com/LayerManager/layman/issues/1126) Endpoint [Workspace Maps](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#workspace-maps) (with methods GET, POST, DELETE) was unified into endpoint [Maps](doc/rest.md#get-maps); `workspace` is now supplied via request query/body parameter instead of URL path.
+- [#1126](https://github.com/LayerManager/layman/issues/1126) Endpoint [GET Workspace Layer Metadata Comparison](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-metadata-comparison) was removed and replaced with endpoint [GET Layer Metadata Comparison](doc/rest.md#get-layer-metadata-comparison) to use UUID-based URL `/rest/layers/{uuid}/metadata-comparison` instead of workspace&name-based URL.
 
 ## v2.1.0
  2025-05-02
@@ -187,7 +188,7 @@
 - [#165](https://github.com/LayerManager/layman/issues/165) POST Workspace [Layers](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#post-workspace-layers)/[Maps](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#post-workspace-maps) and PATCH Workspace [Layer](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#patch-workspace-layer)/[Map](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#patch-workspace-map) saves [role names](doc/models.md#role) mentioned in `access_rights.read` and `access_rights.write` parameters into [prime DB schema](doc/data-storage.md#postgresql).
 - [#165](https://github.com/LayerManager/layman/issues/165) Many requests respect roles in access rights:
   - [GET](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer)/[PATCH](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#patch-workspace-layer)/[DELETE](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#delete-workspace-layer) Workspace Layer
-  - GET Workspace Layer [Thumbnail](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-thumbnail)/[Style](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-style)/[Metadata Comparison](doc/rest.md#get-workspace-layer-metadata-comparison)/[Chunk](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-chunk)
+  - GET Workspace Layer [Thumbnail](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-thumbnail)/[Style](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-style)/[Metadata Comparison](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-metadata-comparison)/[Chunk](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-chunk)
   - [GET](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-map)/[PATCH](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#patch-workspace-map)/[DELETE](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#delete-workspace-map) Workspace Map
   - GET Workspace Map [File](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-map-file)/[Thumbnail](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-map-thumbnail)/[Metadata Comparison](doc/rest.md#get-workspace-map-metadata-comparison)
   - GET Workspace [Layers](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layers)/[Maps](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-maps)
@@ -288,7 +289,7 @@
   - [POST](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#post-workspace-layers) and [DELETE](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#delete-workspace-layers) Workspace Layers
   - [POST](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#post-workspace-maps) and [DELETE](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#delete-workspace-maps) Workspace Maps
   - requests to [WMS](doc/endpoints.md#web-map-service) and [WFS](doc/endpoints.md#web-feature-service) endpoints
-- [#868](https://github.com/LayerManager/layman/issues/868) Responses to [GET Workspace Layer Metadata Comparison](doc/rest.md#get-workspace-layer-metadata-comparison) and [GET Workspace Map Metadata Comparison](doc/rest.md#get-workspace-map-metadata-comparison) do not respect [HTTP X-Forwarded headers](doc/client-proxy.md#x-forwarded-http-headers) of the request intentionally, in order to keep URLs in canonical form.
+- [#868](https://github.com/LayerManager/layman/issues/868) Responses to [GET Workspace Layer Metadata Comparison](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-metadata-comparison) and [GET Workspace Map Metadata Comparison](doc/rest.md#get-workspace-map-metadata-comparison) do not respect [HTTP X-Forwarded headers](doc/client-proxy.md#x-forwarded-http-headers) of the request intentionally, in order to keep URLs in canonical form.
 - [#868](https://github.com/LayerManager/layman/issues/868) Relations between map and [internal layers](doc/models.md#internal-map-layer) are updated in `map_layer` table when calling [POST Workspace Maps](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#post-workspace-maps), [PATCH Workspace Map](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#patch-workspace-map), [DELETE Workspace Map](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#delete-workspace-map), and [DELETE Workspace Maps](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#delete-workspace-maps).
 - [#901](https://github.com/LayerManager/layman/issues/901) Endpoints [POST](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#post-workspace-maps) and [PATCH](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#patch-workspace-map) accept map compositions in version `3.x.x`.
 - [#927](https://github.com/LayerManager/layman/issues/927) SLD styles are internally published to GeoServer with [`raw`](https://docs-archive.geoserver.org/2.21.x/en/user/rest/api/styles.html#raw) parameter set to `True`.
@@ -1093,7 +1094,7 @@ There is a critical bug in this release, posting new layer breaks Layman: https:
 ### Changes
 - Both [PATCH Workspace Layer](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#patch-workspace-layer) and [PATCH Workspace Map](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#patch-workspace-map) automatically update also CSW metadata records.
 - Use absolute URLs in responses of Layer and Map endpoints
-- Add [GET Workspace Layer Metadata Comparison](doc/rest.md#get-workspace-layer-metadata-comparison) and [GET Workspace Map Metadata Comparison](doc/rest.md#get-workspace-map-metadata-comparison) endpoints. 
+- Add [GET Workspace Layer Metadata Comparison](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer-metadata-comparison) and [GET Workspace Map Metadata Comparison](doc/rest.md#get-workspace-map-metadata-comparison) endpoints. 
 - Add [`revision_date`](doc/metadata.md#revision_date) metadata property
 - Add `metadata.comparison_url` to [GET Workspace Layer](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-layer) and [GET Workspace Map](https://github.com/LayerManager/layman/blob/v2.1.0/doc/rest.md#get-workspace-map) responses.
 - Upgrade Layman test client to version 1.1.1

doc/client-proxy.md

@@ -82,4 +82,4 @@ Currently, value of X-Forwarded headers affects following URLs:
   * [WFS endpoints](endpoints.md#web-feature-service)
     * all operations URLs
 
-Values of X-Forwarded headers does not affect responses of [GET Workspace Layer Metadata Comparison](rest.md#get-workspace-layer-metadata-comparison) and [GET Workspace Map Metadata Comparison](rest.md#get-workspace-map-metadata-comparison) intentionally, in order to keep URLs in canonical form.
+Values of X-Forwarded headers does not affect responses of [GET Layer Metadata Comparison](rest.md#get-layer-metadata-comparison) and [GET Workspace Map Metadata Comparison](rest.md#get-workspace-map-metadata-comparison) intentionally, in order to keep URLs in canonical form.

doc/rest.md

@@ -9,7 +9,7 @@
 |Layer Thumbnail|`/rest/layers/<uuid>/thumbnail`|[GET](#get-layer-thumbnail)| x | x | x |
 |Layer Style|`/rest/layers/<uuid>/style`|[GET](#get-layer-style)| x | x | x |
 |Layer Chunk|`/rest/layers/<uuid>/chunk`|[GET](#get-layer-chunk)| [POST](#post-layer-chunk) | x | x |
-|Workspace Layer Metadata Comparison|`/rest/workspaces/<workspace_name>/layers/<layername>/metadata-comparison`|[GET](#get-workspace-layer-metadata-comparison) | x | x | x |
+|Layer Metadata Comparison|`/rest/layers/<uuid>/metadata-comparison`|[GET](#get-layer-metadata-comparison) | x | x | x |
 |Maps|`/rest/maps`|[GET](#get-maps)| [POST](#post-maps) | x | [DELETE](#delete-maps) |
 |[Map](models.md#map)|`/rest/maps/<uuid>`|[GET](#get-map)| x | [PATCH](#patch-map) | [DELETE](#delete-map) |
 |Map Thumbnail|`/rest/maps/<uuid>/thumbnail`|[GET](#get-map-thumbnail)| x | x | x |
@@ -330,7 +330,7 @@ JSON object with following structure:
   - *identifier*: String. Identifier of metadata record in CSW instance.
   - *record_url*: String. URL of metadata record accessible by web browser, probably with some editing capabilities.
   - *csw_url*: String. URL of CSW endpoint. It points to CSW endpoint of Micka.
-  - *comparison_url*: String. URL of [GET Workspace Layer Metadata Comparison](#get-workspace-layer-metadata-comparison).
+  - *comparison_url*: String. URL of [GET Layer Metadata Comparison](#get-layer-metadata-comparison).
   - *status*: Status information about metadata import and availability. See [GET Layer](#get-layer) **wms** property for meaning.
   - *error*: If status is FAILURE, this may contain error object.
 - **access_rights**:
@@ -505,7 +505,7 @@ Content-Type: `application/json`
 HTTP status code 200 if chunk was successfully saved.
 
 
-### GET Workspace Layer Metadata Comparison
+### GET Layer Metadata Comparison
 Get comparison of metadata properties among Layman, CSW, WMS and WFS.
 
 #### Request

src/layman/layer/__init__.py

@@ -39,9 +39,9 @@ def get_layer_patch_keys():
 
 from ..common import InternalSourceTypeDef
 from .rest_layer_chunk import bp as layer_chunk_bp
+from .rest_layer_metadata_comparison import bp as layer_metadata_comparison_bp
 from .rest_layer_thumbnail import bp as layer_thumbnail_bp
 from .rest_layer_style import bp as layer_style_bp
-from .rest_workspace_layer_metadata_comparison import bp as workspace_layer_metadata_comparison_bp
 from .rest_layers import bp as layers_bp
 from .rest_layer import bp as layer_bp
 
@@ -52,11 +52,11 @@ def get_layer_patch_keys():
         'name': PUBLICATION_TYPE_NAME,
         'rest_path_name': LAYER_REST_PATH_NAME,
         'workspace_blueprints': [  # blueprints to register
-            workspace_layer_metadata_comparison_bp,
         ],
         'blueprints': [  # blueprints to register
             layers_bp,
             layer_bp,
+            layer_metadata_comparison_bp,
             layer_thumbnail_bp,
             layer_style_bp,
             layer_chunk_bp,

src/layman/layer/micka/csw.py

@@ -46,8 +46,7 @@ def get_layer_info(workspace, layername, *, x_forwarded_items=None):
                 'identifier': muuid,
                 'csw_url': settings.CSW_PROXY_URL,
                 'record_url': common_util.get_metadata_url(layer.uuid, url_type=common_util.RecordUrlType.BASIC),
-                'comparison_url': url_for('rest_workspace_layer_metadata_comparison.get', workspace=workspace, layername=layername,
-                                          x_forwarded_items=x_forwarded_items),
+                'comparison_url': url_for('rest_layer_metadata_comparison.get', uuid=layer.uuid, x_forwarded_items=x_forwarded_items),
             }
         }
     return {}

src/layman/layer/rest_layer_metadata_comparison.py

@@ -0,0 +1,27 @@
+from flask import Blueprint, current_app as app, g, jsonify
+
+from layman.authn import authenticate
+from layman.authz import authorize_uuid_publication_decorator
+from layman.util import check_uuid_decorator
+from . import util, LAYER_REST_PATH_NAME, LAYER_TYPE
+from .layer_class import Layer
+
+bp = Blueprint('rest_layer_metadata_comparison', __name__)
+
+
+@bp.before_request
+@check_uuid_decorator
+@authenticate
+@authorize_uuid_publication_decorator(expected_publication_type=LAYER_TYPE)
+def before_request():
+    pass
+
+
+@bp.route(f"/{LAYER_REST_PATH_NAME}/<uuid>/metadata-comparison", methods=['GET'])
+def get(uuid):
+    app.logger.info(f"GET Layer Metadata Comparison, actor={g.user}")
+
+    layer = Layer(uuid=uuid)
+    md_props = util.get_metadata_comparison(layer)
+
+    return jsonify(md_props), 200

src/layman/layer/rest_workspace_layer_metadata_comparison.py

@@ -180,8 +180,8 @@ def test_post_layers_simple():
     md_record_url = f"http://micka:80/record/basic/m-{layeruuid}"
     assert layer_info['metadata']['record_url'].replace("http://localhost:3080", "http://micka:80") == md_record_url
     with app.app_context():
-        assert layer_info['metadata']['comparison_url'] == test_util.url_for_external('rest_workspace_layer_metadata_comparison.get',
-                                                                                      workspace=workspace, layername=layername)
+        assert layer_info['metadata']['comparison_url'] == test_util.url_for_external('rest_layer_metadata_comparison.get',
+                                                                                      uuid=layeruuid)
     assert 'id' not in layer_info.keys()
     assert 'type' not in layer_info.keys()
 

src/layman/layer/rest_workspace_test.py

@@ -122,8 +122,7 @@ class TestResponsesClass:
         'geodata_type': 'vector',
         'file': {'paths': [f'layers/{layer_uuid}/input_file/{layer_uuid}.geojson'],
                  },
-        'metadata': {'comparison_url': f'http://{settings.LAYMAN_PROXY_SERVER_NAME}/rest/workspaces/{workspace}/layers/{publication}/'
-                                       f'metadata-comparison',
+        'metadata': {'comparison_url': f'http://{settings.LAYMAN_PROXY_SERVER_NAME}/rest/layers/{layer_uuid}/metadata-comparison',
                      'csw_url': f'{settings.CSW_PROXY_URL}',
                      'identifier': f"m-{layer_uuid}",
                      'record_url': None},

src/layman/rest_responses_test.py

@@ -80,7 +80,7 @@
                                                         'rest_layers.delete',
                                                         layer_keys_to_check,
                                                         'sample/layman.layer/small_layer.geojson',
-                                                        'rest_workspace_layer_metadata_comparison.get',
+                                                        'rest_layer_metadata_comparison.get',
                                                         'rest_layer_chunk.post',
                                                         ),
                          None: PublicationTypeDef('publicationname',
@@ -492,7 +492,13 @@ def get_workspace_publication_metadata_comparison(publication_type, workspace, n
 
     publication_type_def = PUBLICATION_TYPES_DEF[publication_type]
     with app.app_context():
-        r_url = url_for(publication_type_def.get_workspace_metadata_comparison_url, **{publication_type_def.url_param_name: name}, workspace=workspace)
+        if publication_type == LAYER_TYPE:
+            uuid = layman_util.get_publication_uuid(workspace, publication_type, name)
+            r_url = url_for(publication_type_def.get_workspace_metadata_comparison_url, uuid=uuid)
+        else:
+            r_url = url_for(publication_type_def.get_workspace_metadata_comparison_url,
+                            **{publication_type_def.url_param_name: name},
+                            workspace=workspace)
     response = requests.get(r_url, headers=headers, timeout=HTTP_TIMEOUT)
     raise_layman_error(response)
     return response.json()