Skip to content

Conflict error fix#129

Merged
turetske merged 6 commits into
PelicanPlatform:mainfrom
turetske:ConflictErrorFix
Feb 9, 2026
Merged

Conflict error fix#129
turetske merged 6 commits into
PelicanPlatform:mainfrom
turetske:ConflictErrorFix

Conversation

@turetske

Copy link
Copy Markdown
Collaborator

Closes #128
Closes #47

This fix implements an _ls_from_https calls which ensures that any ls call from https will be redirected to the appropriate collections url.

This code also reduces the number of director calls to get the collections url by caching that information as part of the namespace-info structure to be reused when we have prefix matches.

Comment thread src/pelicanfs/core.py Outdated
Comment thread test/test_readme_examples.py
… url

-- Created an _ls_from_https which monkeypatches the http file system's _ls
-- Now ALL list calls will go to a collections URL
-- Also added cacheing of the collections URL in the director response to prevent constant director calls
-- Added tests which ensure the readme examples work
-- Added tests of 'get' functionality
-- Updated namespace_info tests to clear the namespace info cache between tests
@turetske turetske added the bug Something isn't working label Jan 30, 2026

@patrickbrophy patrickbrophy left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@turetske turetske merged commit 990eaa0 into PelicanPlatform:main Feb 9, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Listings not always using the collections url Reduce director calls for get_dirlist_url

2 participants