816429 - Can't access DeviceStorage in Bluetooth app

Status: RESOLVED DUPLICATE of bug 814323

(Firefox OS Graveyard :: Gaia::Bluetooth, defect)

Description

[Eric Chou [:ericchou] [:echou]]

Originally, when users try to send a file, a prompt would be shown to ask for user's approval to access device storage. After Bug 814294 landed, no prompt appears even if we share a file via Bluetooth, and we will get a "Bluetooth.getFile: SecurityError" error message from logcat.

Comment 1

[Eric Chou [:ericchou] [:echou]]

cc Gregor, assignee of bug 814294

Hi Gregor,

  I'm not sure how to make permission prompt to be shown while Bluetooth app trying to access DeviceStorage for Bluetooth file-sharing. Could you help us?

  Thanks in advance.

Eric

Comment 2

[Gregor Wagner [:gwagner]]

You need the right permission for it. Currently I see for the bluetooth app:
    "device-storage:pictures":{ "access": "readwrite" },
    "device-storage:music":{ "access": "readwrite" }
What media type do you want to access?
Also try a make reset-gaia to make sure you have the current version of the permission DB.

Comment 3

[Gregor Wagner [:gwagner]]

If the right permission is in the manifest, please look at b2g/components/ContentPermissionPrompt.js and let me know what variables (request.type and request.access) you see in the handleExistingPermission function for the failing prompt.

Comment 4

[Dominic Kuo [:dkuo]]

Gregor, we are using navigator.getDeviceStorage('sdcard') in bluetooth app, so what's the current permission we should write?

Comment 5

[Gregor Wagner [:gwagner]]

(In reply to Dominic Kuo [:dkuo] from comment #4)
> Gregor, we are using navigator.getDeviceStorage('sdcard') in bluetooth app,
> so what's the current permission we should write?

Then you have to add the sdcard permission to the manifest:
"device-storage:sdcard":{ "access": "readwrite" }

Comment 6

[Gregor Wagner [:gwagner]]

Does this work now?

Comment 7

[Eric Chou [:ericchou] [:echou]]

(In reply to Gregor Wagner [:gwagner] from comment #6)
> Does this work now?

Yes, it works. I think Ian or Dominic will send a patch tomorrow.

Thank you, Gregor.

Comment 8

[Dominic Kuo [:dkuo]]

Ian has sent a patch includes that change we mention above
https://github.com/mozilla-b2g/gaia/pull/6706

Re-assigning to Ian, thanks.

Comment 9

[Chris Lee [:clee]]

What's the reason we need to ask the user for permission here?  Please help clarify, thanks.

Comment 10

[Gregor Wagner [:gwagner]]

(In reply to Chris Lee [:clee] from comment #9)
> What's the reason we need to ask the user for permission here?  Please help
> clarify, thanks.

We won't ask the user for permission in this scenario.
We had a bug in our permission implementation so that missing permissions in the manifest file resulted in prompting and not in an automatic deny. So the prompt was wrong in the first place.

Comment 11

[Eric Chou [:ericchou] [:echou]]

This has been resolved in Bug 814323 by Ian.