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

Permission Denial: writing android.support.v4.content.FileProvider uri content://org.cryptomator.beta.fileprovidor/decrypted/filename.ext #51

Closed
earthsound opened this issue Apr 5, 2017 · 4 comments
Assignees
Labels
misc:beta state:has-workaround There is a known workaround for the described problem type:bug Something isn't working

Comments

@earthsound
Copy link

earthsound commented Apr 5, 2017

Basic Info

This is a bug report / question or something else.

I'm using Android in version: 6.0.1

I'm running Cryptomator in version: 0.4.1

Description

I cannot save or merge a kdbx file back to Cryptomator vault.

Steps to reproduce:

  1. Install Dropbox, Keepass2Android, and & Cryptomator
  2. In Cryptomator, create new vault within Dropbox
  3. Upload kbdx file into vault
  4. Open kdbx file from within Cryptomator > should prompt to open with KeePass2Android
  5. Edit file > Save

I receive this error:

Could not save to remote: 
Permission Denial: writing android.support.v4.content.FileProvider uri content://org.cryptomator.beta.fileprovidor/decrypted/filename.kdbx from pid=26855, uid=10261 requires the provider be exported, or grantUriPermission(). Save again or use the Synchronize menu when remote connection is available again. 

Attempting to synchronize results in the same error message minus the last sentence.


Is this an issue with KP2A, Cryptomator, a combination of the two, or limitations with the OS?

On a side note, if I upload an RTF file into a cryptomator vault and edit it with ES File Explorer's "ES Note Editor," when I save it I get no error but going back to the vault shows that the file wasn't changed.

Did I miss reading somewhere that this it's read only on Android?

@tobihagemann tobihagemann added state:to-be-confirmed We are trying to confirm the reported behavior type:bug Something isn't working labels Apr 5, 2017
@earthsound
Copy link
Author

I opened a ticket at KP2A's CodePlex site and the dev answered:

It's an issue with Cryptomator that they do not serve their vault contents through the Storage Access Framework as a document provider.
Please see #35

If Cryptomator on Android doesn't implement SAF, how are we to use it for anything other than reading files?

@tobihagemann
Copy link
Member

Thanks for the additional information! Just saw that we've stated in #50 (comment) that we're going to add SAF support.

@markuskreusch How should we manage this issue? Would this bug/error remain for Android <5.0 or is there something else that we can do about it? Should we update #35 somehow so that it reflects our plans?

@markuskreusch markuskreusch added misc:beta and removed state:to-be-confirmed We are trying to confirm the reported behavior labels Apr 6, 2017
@markuskreusch
Copy link
Contributor

markuskreusch commented Apr 6, 2017

#50 is related to supporting the SAF as backend for storing our encrypted files. That is different from making the vaults contents accessible via the SAF. This is what #35 is about. I think #35 already states what we plan. In general this is not really a bug but only caused by the SAF feature missing during the beta.

#6, #15, #40 all seems related to this. I will have a look at that and consolidate those issues if possible.

We will adress this. Not sure about the possibilities in Android < 5.0.

Workaround
Do not save the file but create a copy, edit it and then add this copy back to the vault.

@markuskreusch markuskreusch added the state:has-workaround There is a known workaround for the described problem label Apr 6, 2017
@markuskreusch markuskreusch self-assigned this Apr 6, 2017
@markuskreusch markuskreusch added this to the 0.x milestone Apr 6, 2017
@markuskreusch
Copy link
Contributor

Closing this in favor of #15

@markuskreusch markuskreusch removed this from the 0.x milestone May 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
misc:beta state:has-workaround There is a known workaround for the described problem type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants