Skip to content
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

[feature] change the invalid scheme xmldb:exist: to xmldb+exist: #5587

Open
line-o opened this issue Dec 24, 2024 · 2 comments
Open

[feature] change the invalid scheme xmldb:exist: to xmldb+exist: #5587

line-o opened this issue Dec 24, 2024 · 2 comments
Labels
enhancement new features, suggestions, etc.

Comments

@line-o
Copy link
Member

line-o commented Dec 24, 2024

Is your feature request related to a problem? Please describe.

With the status quo all URIs that start with xmldb:exist: are invalid as the colon character : is not allowed within a scheme
(see https://datatracker.ietf.org/doc/html/rfc3986#section-3.1).

Since + character is used widely in other contexts to signal a combination of protocols it might be a good way forward.

I am used to use git+https:, for example.

Describe the solution you'd like

eXist-db to only use valid URIs.

Describe alternatives you've considered

Alternatively, we could drop the exist: portion completely making all URIs just start with xmldb:, if it does not add anything to a URI. So far I have not seen a URI that needs to be xmldb:exist: - xmldb: works as well.
Maybe someone with knowledge of the history behind the decision to recognise xmldb:exist: could chime in here.

Additional context

This is a breaking change whatever direction we will take. So it cannot be back ported to any released major version.
As a benefit, all URIs will be testable and verifiable without having to rely on custom validators and this will allow to further simplify our codebase and reduce a class of errors.

@line-o line-o added the enhancement new features, suggestions, etc. label Dec 24, 2024
@reinhapa
Copy link
Member

@line-o this URI is defined by the XMLDB API here

@adamretter
Copy link
Contributor

Nice idea, but we tried this in the past and gave up - it broke all the things

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement new features, suggestions, etc.
Projects
None yet
Development

No branches or pull requests

3 participants