Closed Bug 1119009 Opened 9 years ago Closed 9 years ago

[Metrics] AppUsage collection shall be for Apps from Marketplace

Categories

(Firefox OS Graveyard :: General, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:2.2+, b2g-v2.2 fixed, b2g-master fixed)

RESOLVED FIXED
2.2 S6 (20feb)
blocking-b2g 2.2+
Tracking Status
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: rdandu, Assigned: marshall)

References

Details

Attachments

(3 files)

AppUsage metric collection shall be restricted to following:
1) Apps from Firefox OS Marketplace, and
2) Preinstalled Apps
App installed from other marketplace's and locally should not be part of the metric collected by Mozilla. 

This will help other marketplace users have choices of privacy.

This requirement enhances the original Bug 1109422 - AppUsage Metric collection enhancements for MonthlyActiveUsers computation.
Assignee: nobody → marshall
The following is proposed for identification of such apps:                                             
1) Firefox OS marketplace: Tracking the "install origin" inside the app database, 'Mozilla marketplace",  

2) Preinstalled: Preinstalled apps will either have an "install origin" of the mozilla marketplace, or their app-type will be "certified".
Marshall, please add an ETA to this bug!

Ravi, If this is a blocker for 2.2, please add blocker justification and work with release owner to get this approved.

Thanks
Hema
Flags: needinfo?(rdandu)
Flags: needinfo?(marshall)
Target Milestone: --- → 2.2 S5 (6feb)
Kevin, 
    Please include this in 2.2. This bug is finetuning the main metric bug 1109422 for Appusage which is method to derive MAU (Monthly Active Users). MAU is the critical metric to track FxOS, decided upon across the products. 

This bug excludes Apps outside of Firefox OS Marketplace.
1)This is to ensure privacy in certain cases. eg: if LGBT community in Russia is hosting app marketplace, we do not want have any info on those apps to ensure additional privacy. 
2)For apps from Firefox OS Marketplace, we already know that they have been installed. Hence they are to be included in the metrics.
blocking-b2g: --- → 2.2?
Flags: needinfo?(rdandu) → needinfo?(khu)
Flags: needinfo?(marshall)
Attachment #8560724 - Flags: review?(dflanagan)
Flags: in-moztrap?(slyu)
Comment on attachment 8560724 [details] [review]
[PullReq] marshall:bug1119009_marketplaceApps to mozilla-b2g:master

r+ for the code changes, but think about:

1) allowing more than one marketplace url, so that partners don't replace our url with their url. Give them a way to add their url.

2) are you sure that testing for type===certified is the right way to detect a pre-installed app?  A lot of developers want to reduce the privilege of the preinstalled apps.  Maybe checking for gaiamobile.org instead?
Attachment #8560724 - Flags: review?(dflanagan) → review+
I agree with David on point 2. Also, since we moved to opt-out now, you might get a lot of data from engineering build that has certified testing apps.

(In reply to David Flanagan [:djf] from comment #5)
> Comment on attachment 8560724 [details] [review]
> [PullReq] marshall:bug1119009_marketplaceApps to mozilla-b2g:master
> 
> r+ for the code changes, but think about:
> 
> 1) allowing more than one marketplace url, so that partners don't replace
> our url with their url. Give them a way to add their url.
> 
> 2) are you sure that testing for type===certified is the right way to detect
> a pre-installed app?  A lot of developers want to reduce the privilege of
> the preinstalled apps.  Maybe checking for gaiamobile.org instead?
Triage is blocking based on the fact that Metrics is a key initiative and high priority.
blocking-b2g: 2.2? → 2.2+
Target Milestone: 2.2 S5 (6feb) → 2.2 S6 (20feb)
I've made the MARKETPLACE_ORIGINS an array to allow partners to add other sources if they need, and I've also added a check for 'gaiamobile.org' in the manifestURL as another case that signifies a preinstalled app
master - https://github.com/mozilla-b2g/gaia/commit/56c9e554bf39ad05e61fd83b33dffabb1957dcfa
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 8567371 [details] [review]
[gaia] marshall:bug1119009_marketplaceApps_v2.2 > mozilla-b2g:v2.2

Only report apps preinstalled apps or installed from marketplace in AppUsageMetrics

[Bug caused by] (feature/regressing bug #): Bug 1119009
[User impact] if declined: We will collect app usage for sideloaded / testing apps

[Testing completed]: Unit and manual
[Risk to taking this patch] (and alternatives if risky): This may filter out some apps we want unintentionally

[String changes made]: None
Attachment #8567371 - Flags: approval-gaia-v2.2?(bbajaj)
Attachment #8567371 - Flags: approval-gaia-v2.2?(bbajaj) → approval-gaia-v2.2+
Depends on: 1137063
clear ni because it's in 2.2.
Flags: needinfo?(khu)
Attached file MozTrap Cases
Flags: in-moztrap?(slyu) → in-moztrap+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: