dandi / dandi-cli

DANDI command line client to facilitate common operations
https://dandi.readthedocs.io/
Apache License 2.0
22 stars 27 forks source link

Authenticate user upon the move command for an embargoed Dandiset #1478

Closed kabilar closed 3 months ago

kabilar commented 3 months ago

I received the error below when trying to rename an asset for an embargoed Dandiset.

Command

dandi move sub-mr261FZ/mr261FZ.tiff sub-mr261FZ/sub-mr261.tiff

Error

2024-08-08 17:08:52,143 [   ERROR] Error 401 while sending GET request to https://api.dandiarchive.org/api/dandisets/001089/: {"detail":"Authentication credentials were not provided."}
2024-08-08 17:08:52,144 [    INFO] Logs saved in /Users/kabilar/Library/Logs/dandi-cli/2024.08.08-22.08.51Z-13496.log
Error: Error 401 while sending GET request to https://api.dandiarchive.org/api/dandisets/001089/: {"detail":"Authentication credentials were not provided."}

We encountered the same error on the LINC project and @aaronkanzer provided a fix which I have submitted in this pull request.

I wasn't able to test these changes to the dandi-cli locally due to some versioneer issue. So please double check this work. Thank you.

kabilar commented 3 months ago

Thank you, John.

github-actions[bot] commented 2 months ago

:rocket: Issue was released in 0.63.1 :rocket: