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

drop_delete with dtoken #196

Open
keblu opened this issue Jul 9, 2021 · 0 comments
Open

drop_delete with dtoken #196

keblu opened this issue Jul 9, 2021 · 0 comments

Comments

@keblu
Copy link

keblu commented Jul 9, 2021

I am using rdrop2 for doing some file and user management for a shinyapp (at least for the moment). When using drop_delete() while setting the dtoken argument to my token (from an encrypted RDS file), the browser popup with the authorization for rdrop2 on dropbox, which is not the behavior expected given that I passed dtoken to the function. This does not do that when passing dtoken to some other drop_*() functions such as drop_upload().

From my diagnosis on why it does that, it seems that the drop_delete() function uses the drop_exists() function internally but does not pass dtoken to the drop_exists() function when calling it. This leads to get_dropbox_token() being called in all cases in the drop_delete() function ( because of the default parameters of drop_exists() ). This happens in other functions such as drop_create(), drop_copy(), drop_move().

For now, I did my own drop_delete2() function changing drop_exists(path) to drop_exists(path, dtoken = dtoken), but it would be nice if this could be done in a future version of the package, at least if my diagnostic is right.

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

No branches or pull requests

1 participant