UDP namespace with @ not allowed/supported?

Open-EO / openeo-web-editor

An interactive and easy to use web-based editor for the OpenEO API.

https://editor.openeo.org

Apache License 2.0

12 stars 17 forks source link

UDP namespace with @ not allowed/supported? #260

Closed jdries closed 1 year ago

jdries commented 2 years ago

I have a UDP in a namespace with an '@'.

The web editor can not properly load it because the @ and everything after it is dropped. Adding it again manually does trigger correct loading. Not sure if @ is an illegal character for namespace or some other problem?

https://editor.openeo.org/?server=https%3A%2F%2Fopeneo.vito.be&namespaces=u%3Adfa678cb9ab17f65d4f025e30fac5e0d90116176e44fd17d703419322747cbbd%40egi.eu&process=EVI%40u%3Adfa678cb9ab17f65d4f025e30fac5e0d90116176e44fd17d703419322747cbbd%40egi.eu

m-mohr commented 2 years ago

The implementation should follow the proposal in PR https://github.com/Open-EO/openeo-api/pull/348

In the proposal, @ plus username is meant to be used for user namespaces so it has a special meaning and is not allowed in user ids, I think.

Regexp for user id: ^[\w\-\.~]+$ Regexp for namespaces: ^@?[\w\-\.~]+$

Has been some time since I worked with user namespaces, but if you follow the PR and the regex, it should work.