-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Validate and monitor backends #8
Comments
Before removing the workarounds we need a way to validate the back-ends during crawling so we can ensure the stored data is valid and doesn't produce problems in the UI. Invalid back-ends need to be rejected (as if the server was not reachable) and the user be informed or at least it must be ensured that invalid back-end don't lead to errors in the Hub UI. |
HTTPS is being enforced since a3b69b4 more than a year ago, this just had been tracked in a different issue, #59. If a URL doesn't start with
This works well as demonstrated by the current attempts to crawl https://openeo.creo.vito.be/.well-known/openeo. The other two points from the list in the initial post refer to things that are not even part of the Hub code anymore (the So from the workarounds perspective, this issue could be closed. But a proper validation of backends would still be good -- before enlisting, but also continuously to detect things like Open-EO/openeo-grassgis-driver#130. I don't reasonably see this in the Hub's scope for now, but we can keep it as an "idea that would be nice to have". |
In the long run, workarounds that were introduced to accommodate incompatible backends should be removed, as only fully spec-compliant backends should be listed in the Hub. See also this conversation.
Once possible, the following should be done:
links
field in/collections
(code)version
prop of the components that are imported from openeo-vue-components (it's not passed so far because backends are under development and therefore their behaviour may differ from the API spec version they report they adhere to -> auto-detection yields better results)list to be continued
The text was updated successfully, but these errors were encountered: