Feature/8727 serve das visual tiles by weited · Pull Request #302 · aodn/ogcapi-java

weited · 2026-07-21T06:38:38Z

No description provided.

Copilot

Pull request overview

Adds support for serving Data Access Service (DAS) “visual tiles” through this OGC API server, including a public collection map-tile proxy endpoint plus “ext” endpoints that expose tiler metadata (products/dates, colormaps, legend) to the frontend.

Changes:

Introduces DasTilerService with upstream error/status mapping and EhCache-backed caching for tiles/products/manifest.
Adds new REST endpoints for DAS tiler “ext” metadata (/api/v1/ogc/ext/tiler/...) and a hand-written collection map tile route that proxies visual tiles.
Adds unit/integration tests covering routing/validation, upstream status mirroring, caching behavior, and ext endpoint responses.

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
server/src/test/java/au/org/aodn/ogcapi/server/tile/TilerExtApiTest.java	Adds SpringBoot tests for tiler ext endpoints (products listing, colormaps, legend).
server/src/test/java/au/org/aodn/ogcapi/server/tile/RestApiTest.java	Extends API tests to cover the visual-tile proxy route validation and upstream status mirroring.
server/src/test/java/au/org/aodn/ogcapi/server/core/service/DasTilerServiceTest.java	Adds unit tests for URL building, header injection, and upstream/network error mapping.
server/src/test/java/au/org/aodn/ogcapi/server/core/service/DasTilerServiceCachingTest.java	Adds a Spring proxy-backed test to validate `@Cacheable` behavior and `cv` staleness checks on cache hits.
server/src/main/resources/application-dev.yaml	Imports optional `.env.dev` for dev-time configuration overrides.
server/src/main/java/au/org/aodn/ogcapi/server/tile/TilerExtApi.java	Adds ext controller exposing DAS tiler metadata and legend proxy endpoints.
server/src/main/java/au/org/aodn/ogcapi/server/tile/RestApi.java	Adds hand-written collection map-tile endpoint proxying DAS visual tiles with validation/bounds checks.
server/src/main/java/au/org/aodn/ogcapi/server/core/service/DasTilerService.java	Implements upstream DAS tiler client with caching, header management, and response forwarding.
server/src/main/java/au/org/aodn/ogcapi/server/core/exception/GlobalExceptionHandler.java	Adds handler to return status/body/content-type for `DasUpstreamException`.
server/src/main/java/au/org/aodn/ogcapi/server/core/exception/DasUpstreamException.java	Adds exception type to carry upstream status/body/content-type to the global handler.
server/src/main/java/au/org/aodn/ogcapi/server/core/configuration/CacheConfig.java	Adds dedicated caches for tiler tiles/products/manifest.
server/pom.xml	Enables preview features for Spring Boot plugin runtime to match compiler preview configuration.
.gitignore	Ignores `.env.dev`.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 24 out of 25 changed files in this pull request and generated 5 comments.

utas-raymondng

Some comment

utas-raymondng · 2026-07-22T05:39:36Z

            long totalHits = getDocumentsCount(vocabsIndexName);
            if (totalHits == 0) {
-                throw new DocumentNotFoundException("No documents found in " + vocabsIndexName);
+                throw new ResourceNotFoundException("No documents found in " + vocabsIndexName);


Any impact to frontend ?

reverted, the old DocumentNotFoundException was not handled at GlobalHandler (not sure if it's intentional or a bug, out of scope), but ResourceNotFounException is handled. The difference might cause different behaviour

utas-raymondng · 2026-07-22T05:40:45Z

        return new RestTemplate(factory);
    }

+    @Bean(name = DAS_TILER_REST_TEMPLATE, defaultCandidate = false)


Is it possible to use the existing as the data access service then you can make your code easier, especially there is a host key need to set in order to avoid firewall block on repeat call

unified rese template for all DAS

utas-raymondng · 2026-07-24T02:17:02Z

Branch outdated

@hidden

Proxies DAS's visual-tile images, product/manifest listing, colormaps, and legend through public ogcapi-java routes so the frontend never needs to talk to DAS directly: a hand-written OGC map-tile route (collectionMapGetTile's generated stub is @hidden and replaced) plus a new /ext/tiler namespace for product discovery, backed by DasTilerService and cached with EhCache.

…eout

…llection assets

utas-raymondng

LGTM

weited temporarily deployed to test July 21, 2026 06:38 — with GitHub Actions Inactive

weited requested a review from Copilot July 21, 2026 06:39

Copilot started reviewing on behalf of weited July 21, 2026 06:39 View session

Copilot AI reviewed Jul 21, 2026

View reviewed changes

Comment thread server/src/main/java/au/org/aodn/ogcapi/server/core/service/das/DasTilerService.java

Comment thread server/src/main/java/au/org/aodn/ogcapi/server/core/exception/DasUpstreamException.java Outdated

weited temporarily deployed to test July 21, 2026 17:48 — with GitHub Actions Inactive

weited force-pushed the feature/8727-serve-das-visual-tiles branch from 6cb9eff to 3977749 Compare July 22, 2026 01:13

weited temporarily deployed to test July 22, 2026 01:13 — with GitHub Actions Inactive

weited had a problem deploying to test July 22, 2026 05:30 — with GitHub Actions Error

weited force-pushed the feature/8727-serve-das-visual-tiles branch from eed458f to 294bdb2 Compare July 22, 2026 05:33

weited temporarily deployed to test July 22, 2026 05:34 — with GitHub Actions Inactive

weited requested a review from Copilot July 22, 2026 05:35

Copilot started reviewing on behalf of weited July 22, 2026 05:35 View session

weited requested a review from utas-raymondng July 22, 2026 05:37

Copilot AI reviewed Jul 22, 2026

View reviewed changes

utas-raymondng requested changes Jul 22, 2026

View reviewed changes

weited force-pushed the feature/8727-serve-das-visual-tiles branch from 294bdb2 to ca3e00b Compare July 22, 2026 23:46

weited temporarily deployed to test July 22, 2026 23:46 — with GitHub Actions Inactive

weited requested a review from utas-raymondng July 22, 2026 23:52

weited temporarily deployed to test July 23, 2026 08:10 — with GitHub Actions Inactive

weited temporarily deployed to test July 24, 2026 01:24 — with GitHub Actions Inactive

weited force-pushed the feature/8727-serve-das-visual-tiles branch from c52621c to 1b4d674 Compare July 24, 2026 03:37

weited had a problem deploying to test July 24, 2026 03:37 — with GitHub Actions Error

weited added 7 commits July 24, 2026 13:40

Enable preview features in Maven configuration

8468c17

Address review: stale-cv cache bypass, unbounded zoom, tile-route tim…

17a924c

…eout

Remove all caching from DasTilerService

6f1e684

Add a dedicated RestTemplate for the DAS tiler to have its own client

fe7ad59

Add RestExtApi for tile endpoints

3171ef3

Add OpenAPI docs for the DAS visual-tile routes

83b897d

weited added 5 commits July 24, 2026 13:40

Unify tile-route error responses and exeption handling

ae92563

Attach DAS credentials on the RestTemplate bean

242895e

Address review, cache DAS product list, fix ext tile error schemas

c423b1b

Split product to dataset and variable, validate dataset via cached co…

f8790c8

…llection assets

Remove unused configuration from test

1b7aa82

weited force-pushed the feature/8727-serve-das-visual-tiles branch from 1b4d674 to 1b7aa82 Compare July 24, 2026 03:41

weited temporarily deployed to test July 24, 2026 03:41 — with GitHub Actions Inactive

utas-raymondng approved these changes Jul 24, 2026

View reviewed changes

utas-raymondng merged commit 6efe5b8 into main Jul 24, 2026
4 checks passed

utas-raymondng deleted the feature/8727-serve-das-visual-tiles branch July 24, 2026 03:50

Uh oh!

Conversation

weited commented Jul 21, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

utas-raymondng left a comment

Choose a reason for hiding this comment

Uh oh!

utas-raymondng Jul 22, 2026

Choose a reason for hiding this comment

Uh oh!

weited Jul 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

utas-raymondng Jul 22, 2026

Choose a reason for hiding this comment

Uh oh!

weited Jul 22, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

utas-raymondng commented Jul 24, 2026

Uh oh!

utas-raymondng left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

weited Jul 22, 2026 •

edited

Loading