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

Work around registry URLs with multiple slashes by outputting paths with single slashes #163

Merged
merged 1 commit into from
Apr 19, 2024

Conversation

nutjob4life
Copy link
Member

🗒️ Summary

Merge this to workaround registry file_ref URLs that have double-slashes in them as reported in #162 and experience by some users.

⚙️ Test Data and/or Report

Before:

$ pds-deep-registry-archive --quiet --site PDS_ATM urn:nasa:pds:cassini_uvis_solarocc_beckerjarmak2023::1.0
$ egrep -c '//data' cassini_uvis_solarocc_beckerjarmak2023_v1.0_*_sip_v1.0.tab
86

After:

$ pds-deep-registry-archive --quiet --site PDS_ATM urn:nasa:pds:cassini_uvis_solarocc_beckerjarmak2023::1.0
$ egrep -c '//data' cassini_uvis_solarocc_beckerjarmak2023_v1.0_*_sip_v1.0.tab
0

♻️ Related Issues

@jordanpadams jordanpadams merged commit 75f130f into main Apr 19, 2024
3 checks passed
@jordanpadams jordanpadams deleted the i162 branch April 19, 2024 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants