UnifiedNlp icon indicating copy to clipboard operation
UnifiedNlp copied to clipboard

Doesn't work with online backends

Open amilopowers opened this issue 9 years ago • 4 comments

I use a hammerhead (Nexus 5) device with OmniROM 6.0.1. I tried it on CopperheadOS as well and it did't work either.

Since I tried Mozillas backend (crashes) and the openbmap one from F-Droid and both fail to get a location fix I guess the problem should be in the middleware....

I cloudln't try locale backends yet.

amilopowers avatar Feb 14 '16 11:02 amilopowers

UnifiedNlp does not properly handle Android 6 permissions. Please check that all permissions are granted to UnifiedNlp and the backends.

mar-v-in avatar Feb 14 '16 18:02 mar-v-in

Oh thanks now it works. Could you write this in the f-droid description?

Btw: do have an idea why Mozilla backend asks for phone permission?

amilopowers avatar Feb 14 '16 19:02 amilopowers

The permission requested by Mozilla backend is READ_PHONE_STATE, which is necessary to determine nearby cell towers (as well as ACCESS_COARSE_LOCATION), due to Android 6 permission groups, this is the same group as CALL permission. You can view the real permissions of an app by using the opening "all permissions" in the menu of an apps permission list.

I am not involved in F-Droid description of UnifiedNlp, as I don't publish there. However this should be fixed in UnifiedNlp (and backends) rather than having a manual work-around. Regarding UnifiedNlp, this will be included in next version of the Self-Check. For backends, the UnifiedNlp API will be updated to allow initial setup procedures (like asking for permissions or required configuration). - I will keep this issue open as a reminder for those things.

mar-v-in avatar Feb 14 '16 19:02 mar-v-in

Thanks for clarifying. Weird is just that the OpenBmap backend doesn't asks for that... So I guess it doesn't check cell towers even though the data is collected by the project.

amilopowers avatar Feb 14 '16 20:02 amilopowers