Skip to content

Allow a path of rev: or id: for get comands - #56

Closed
blakelauer wants to merge 1 commit into
dropbox:masterfrom
blakelauer:master
Closed

blakelauer wants to merge 1 commit into
dropbox:masterfrom
blakelauer:master

Conversation

@blakelauer

Copy link
Copy Markdown

The API allows for the download (get) command to use "path": "rev:a1c10ce0dd78" or "path": "id:a4ayc_80_OEAAAAAAAAAYa" but the validatePath function adds "/" to the path if it is not there. This
results in "path": "/rev:a1c10ce0dd78" and an error.

This change stops the "/" from being added if the path begins with "rev:" or "id:". This allows the command dbxcli get rev:a1c10ce0dd78 file.ext to run and download a specific revision of a file.

The API allows for the download (get) command to use "path":
"rev:a1c10ce0dd78" or "path": "id:a4ayc_80_OEAAAAAAAAAYa" but the
validatePath function adds "/" to the path if it is not there. This
results in "path": "/rev:a1c10ce0dd78" and an error.
@diwakergupta

Copy link
Copy Markdown
Collaborator

Thanks @blakelauer ! Can you also update the documentation to reflect this new capability?

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@iminet

iminet commented Jul 17, 2026

Copy link
Copy Markdown

Any update on this? I'm really missing this feature and still awaiting for implementation.
Thanks in advance.

@AndreyVMarkelov

Copy link
Copy Markdown
Contributor

Will be into the next release

@AndreyVMarkelov

Copy link
Copy Markdown
Contributor

Thanks for the contribution! This functionality has since been implemented in #359 and is now available in dbxcli. The get command supports Dropbox references such as rev:, id:, and ns:. Since the requested functionality is now available through a newer implementation, I’m closing this PR.

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.

5 participants