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

Add option to verify checksum without requiring write access to remote objects #11

Open
ghuls opened this issue Dec 9, 2024 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@ghuls
Copy link

ghuls commented Dec 9, 2024

Add option to verify checksum without requiring write access to remote objects and without requiring to upload new data.

I wanted to check some data uploaded by another user, which I can read, but with the current mango-ingest, I can't verify the checksums as it requires write permissions to remote object by default.

Trying to run the --dry-run option with --verify-checksum does not work for just checking the checksum as it will just print dry-run: would upload /path/to/file

$ mango_ingest.py -r -p local_dir/ -d /irods --verify-checksum --verbose --no-watch --dry-run
@paulborgermans
Copy link
Collaborator

Indeed, checksum operations require write access to the object by the user executing the code. That is not something that I/we can change as it is the way the irods backend works. Maybe a stricter checking of access rights should be added?

@paulborgermans paulborgermans self-assigned this Dec 18, 2024
@paulborgermans paulborgermans added the enhancement New feature or request label Dec 18, 2024
@ghuls
Copy link
Author

ghuls commented Dec 20, 2024

I don't see why it shouldn't be possible to retrieve a precalculated checksum from iRODS as a user with read only rights. If that is really not possible it is something that should be fixed in the iRODS backend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants