support new CDSE dev backend URL in test suite#497
Conversation
|
maybe it's better in this case to write a regex instead of trying to have an exact list ? |
@soxofaan, how much freedom should this regex then have? Don't you have the risk of allowing a non-existing backend URL when not using these allow lists? |
|
@pvbouwel do you have a suggestion for a regex to avoid hardcoding CDSE dev hostnames like |
|
So the only form that should be supported for our cloud environments is For OpenEO endpoint service is "openeo" So openeo.dev.warsaw.openeo.dataspace.copernicus.eu should anyway be removed as there should not be a use case that cannot be served by openeo.dev.waw3-1.openeo-int.v1.dataspace.copernicus.eu and that one should be preferred. |
|
so ideally the regex should be something like elif re.fullmatch(r"openeo\.dev\.([a-z0-9-]+)\.openeo-int\.v1\.dataspace\.copernicus\.eu", hostname): |
|
@soxofaan - Can this PR be merged? |
1e8f925 to
2033588
Compare
|
done |
I was recently informed by Peter (can't tag him here) that
https://openeo.dev.waw3-1.openeo-int.v1.dataspace.copernicus.eu/is the new go-to URL for CDSE dev. However, the test suite didn't accept this one yet.