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

Data on sd-card only readable for other android apps under android 6.0x #289

Closed
JSoko opened this issue Sep 24, 2016 · 3 comments
Closed

Comments

@JSoko
Copy link
Member

JSoko commented Sep 24, 2016

Actual behaviour

If you use the storagepath on sd, other apps, p.e. keepass2Android or Gallery can only access to the files readonly. Only Nextcloud itself can write to its directories.

Path to sd is /storage/9016-XXXX/Android/data/com.nextcloud.android.beta/files
Path to memory is /storage/emulated/0/Android/data/com.nextcloud.android.beta/files

I think this issue is by security design of android 6.0x, but makes the storing to sd-card nearly useless, because, you can't change or delete files using other apps.

Expected behaviour

Also other apps should be able to change or delete files in the Nextcloud sd-directory

Steps to reproduce

  1. try to access to a file witch is stored in the Nextcloud Directory on the sd card with gallery or keepass2android
  2. you can read but not write or delete the files with other apps

Environment data

Android version:
6.01
Device model:
Samsung Galaxy S7
Stock or customized system:
Stock
Nextcloud app version:
beta 20160922
Nextcloud server version:
10.0.16

Logs

Web server error log

no error

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Sep 26, 2016

Hi @JSoko yes this is afaik by security design since you are referring to the app's data directory which is protected by Android. As soon as we have implemented #285 step 2 (step one is a instant upload replacement, step 2 would be a "sync any kind of folder pair") this problem should be solved. But again, the app's data directory is protected.

Besides we do have a read only document provider for now but might implement a write-access document provider so other apps could use this to gain read/write access to these files.

@su-ex
Copy link

su-ex commented Dec 19, 2016

It would be great to have the possibility to choose the sync directory just in a way it's possible with the Desktop client. Furthermore it seems to not be possible for the current beta to write it's directory somehow, all downloads fail.
Maybe wrong place, but new instant upload does nothing also.
Step two: If I think of the desktop client, it's possible to disable sync of directories, with the app it would be great to have this inversed, meaning select those which will be synced and the others won't because there's usually less space than with a desktop.

@tobiasKaminsky
Copy link
Member

As in other issues also requested we will change the sd storage path to ".../media/..." to allow other apps to access to the downloaded files.

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

4 participants