Skip to content

Drop "public" UDP feature? #379

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

Open
soxofaan opened this issue Mar 6, 2025 · 6 comments
Open

Drop "public" UDP feature? #379

soxofaan opened this issue Mar 6, 2025 · 6 comments

Comments

@soxofaan
Copy link
Member

soxofaan commented Mar 6, 2025

This is a question that came up while working on a supporting refactor for Open-EO/openeo-aggregator#125. I was slowed down quite a bit with this non-standard "public" UDP feature:

  • user johndoe saves a UDP with public=True
  • other users can list this UDP under GET /processes/u:johndoe
  • other users can load this UDP with namespace u:johndoe

also described in python client docs at https://open-eo.github.io/openeo-python-client/cookbook/udp_sharing.html

I think we nowadays settled on UDP sharing through URLs (e.g. with the official remote process definition extension) instead of this non-standard "public" flag.

Is it still useful/necessary to keep this non-standard "public" UDP feature alive?
Or can we clean that up to simplify the related code paths?

@soxofaan
Copy link
Member Author

soxofaan commented Mar 6, 2025

on CDSE I still see related requests happen, for example :

  • GET /openeo/processes/u:3e24....500e576574/BIOPAR
  • GET /openeo/1.2/processes/u:3e....e576574/biopar_8bands_060325_v2

@soxofaan
Copy link
Member Author

soxofaan commented Mar 6, 2025

I added warning log on usage of these non-standard urls to better track this

@jdries
Copy link
Contributor

jdries commented Mar 12, 2025

The EOPlaza docs and samples point towards this:
https://docs.terrascope.be/Developers/EOplaza/Services.html#develop-an-openeo-algorithm

So that requires a proper update and retraining the trainers on the preferential way to publish UDP.

@soxofaan
Copy link
Member Author

@Pratichhya also mentioned that this feature is still in use can not just be dropped yet.

@soxofaan
Copy link
Member Author

But also to consider here is this related, more general problem:

We're planning to push users more towards federated openEO backends, but in that context we at most have hacky/partial support for UDPs (in the /process_graphs endpoint sense) even for the standardized part of that subsystem.

@Pratichhya
Copy link

Ahh thank you @soxofaan will be following it.
@jdries thank you for pointing that out, will update the documention on limiting the use of 'public' flag

pvbouwel pushed a commit to pvbouwel/openeo-python-driver that referenced this issue Mar 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants