Closed Bug 1060178 Opened 6 years ago Closed 5 years ago

[ALA] check the settings DB for the location precision

Categories

(Firefox OS Graveyard :: General, defect)

All
Gonk (Firefox OS)
defect
Not set

Tracking

(feature-b2g:2.2+)

RESOLVED WONTFIX
feature-b2g 2.2+

People

(Reporter: marta, Assigned: huseby)

References

Details

(Keywords: privacy)

Attachments

(1 file, 1 obsolete file)

Look at the settings DB for global settings of ALA on/off and the precision of location returned (or fake location).
Blocks: 1060181
Assignee: nobody → marta
Summary: ALA: check the settings DB for the location precision → [ALA] check the settings DB for the location precision
Keywords: privacy
OS: Linux → Gonk (Firefox OS)
Hardware: x86_64 → All
QA Whiteboard: STATUS: engineering design
Priority: -- → P4
QA Whiteboard: STATUS: engineering design → STATUS: not started
Attached patch Bug1060178.patch (obsolete) — Splinter Review
Per the comments on:
https://github.com/mozilla/gecko-dev/pull/47
https://github.com/mozilla/gecko-dev/pull/48
https://github.com/mozilla/gecko-dev/pull/49

gecko-dev cannot accept pull requests, so please stop opening them - it's not the correct workflow for development work here.

The patch you have attached is sufficient on it's own - no need to open a PR (you can of course continue to work in your own user fork).
Flags: needinfo?(daniel.marecik)
Reading global settings from the settings DB is ready.

Following tasks are required to create full feature functionality:
- reading app exception list (from setting DB),
- reading list of unblurred app (from a predefined file or setting DB).

These tasks should be created as separate bugs.
Flags: needinfo?(daniel.marecik)
I will not create PR.
Assignee: marta → daniel.marecik
QA Whiteboard: STATUS: not started → STATUS: ready for review
Priority: P4 → --
Depends on: 1063610
Daniel, as have discussed during our call earlier this week, and as showed in other bugs there should be one DB that contains settings for all of the apps:
<ala.global.precise>  = true
<ala.fmd.precise> = true
<ala.weather.blurr> = 5
<ala.netflix.fake> = [51.2345;14.133]
Attached patch Bug1060178.patchSplinter Review
Attachment #8484899 - Attachment is obsolete: true
Reading global settings from the settings DB is ready.
Reading app exception list from the settings DB is ready.
Reading list of unblurred app from the settings DB is ready.

Global settings keys and values:
geolocation.blur.type = 1 (Precise) / 2 (Blur) / 3 (Custom)
geolocation.blur.radius = 6
geolocation.blur.coords = "@51.1234,14.1234"

App exception list:
geolocation.exceptions = { "app//app1.gaiamobile.org/manifest.webapp": { "type": 1 },
     "app//app2.gaiamobile.org/manifest.webapp": { "type": 2, "radius": 5 },
     "app//app3.gaiamobile.org/manifest.webapp": { "type":3, "coords": "@51.1234,14.1234" }}

List of unblurred app: 
geolocation.unblured = ["app//app5.gaiamobile.org/manifest.webapp",
    "app//app6.gaiamobile.org/manifest.webapp",
    "app//app7.gaiamobile.org/manifest.webapp"]
Blocks: 1068044
Blocks: 1069915
Blocks: 1071042
Assignee: daniel.marecik → huseby
See Bug 1073419
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
feature-b2g: --- → 2.2+
You need to log in before you can comment on or make changes to this bug.