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

[DMS] Rationalize the single/double slash in the storage plugins #4744

Closed
chaen opened this issue Sep 18, 2020 · 5 comments
Closed

[DMS] Rationalize the single/double slash in the storage plugins #4744

chaen opened this issue Sep 18, 2020 · 5 comments
Assignees
Labels
Milestone

Comments

@chaen
Copy link
Contributor

chaen commented Sep 18, 2020

Currently we have quite complicated and duplicated code to add or remove double slashes in the storage plugins.
For historical reasons, because SRM was the first, the default in DIRAC is actually against the URI RFC, and we have to do tricks to add the double slash for protocol that wants it.
In practice, most of them supports double and single slash, some are strict with double, and some are strict with single. Those strict with single are SRM (which is going to disappear) and Echo.
This should be factorized, and the default made such that we get a double slash, since the newer protocols insist on having it.

@chaen chaen added the DMS label Sep 18, 2020
@chaen chaen self-assigned this Sep 18, 2020
@fstagni
Copy link
Contributor

fstagni commented Dec 16, 2020

Any target release for this? Any deadline?

@chaen
Copy link
Contributor Author

chaen commented Dec 16, 2020

hum, make it v7r2, since it should be transparent

@fstagni
Copy link
Contributor

fstagni commented Aug 18, 2022

Is the above mentioned PR (which is merged) enough to close this issue?

@chaen
Copy link
Contributor Author

chaen commented Aug 18, 2022

nop :(

@chaen
Copy link
Contributor Author

chaen commented Dec 13, 2022

After digging more into it, won't fix. It's actually more work to fix it than to maintain it, especially that gfal2 does similar things under the hood.

@chaen chaen closed this as completed Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants