You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I propose to remove the default hardcoded version and retrieve it from Rasdaman or ODC.
For the provider instead, we should firstly check if there is one available and if not, set the default one.
Additionally, it seems that maximum 4 providers are allowed.
@aljacob what do you think about this limitation? Should we allow more than 4?
@pierocampa I tag you here just for your interest.
The text was updated successfully, but these errors were encountered:
The version should keep the version of the collection, when specified.
The "Eurac EO WCS" "host (not the provider) should really be factored out to a configuration file.
The provider, which is indeed better taken from the collection's metadata, is however not overwritten now by the server, so no need to act imho. Just the host' is.
As a final note: this ticket exposes a code duplication in the CollectionsApiController class, see describeCollection() and loadWcpsCollections() methods.
I would propose to keep this "v1" as default value in case an input collection does not specify a version, while not overwriting existing versioning. This means that stac_extensions will have to include version (as the currently implemented STAC version is 0.9.0 afaik).
Currently in the exposed metadata the version is hardcoded here:
openeo-spring-driver/src/main/java/org/openeo/spring/api/CollectionsApiController.java
Line 797 in 4f469fa
and the provider has a default value here:
openeo-spring-driver/src/main/java/org/openeo/spring/api/CollectionsApiController.java
Line 1800 in 4f469fa
I propose to remove the default hardcoded version and retrieve it from Rasdaman or ODC.
For the provider instead, we should firstly check if there is one available and if not, set the default one.
Additionally, it seems that maximum 4 providers are allowed.
@aljacob what do you think about this limitation? Should we allow more than 4?
@pierocampa I tag you here just for your interest.
The text was updated successfully, but these errors were encountered: