Android Telemetry planning #2421

ljbade · 2015-09-25T01:38:16Z

For the v0.2.0 release we plan to implement metrics.

This ticket is for planning what tasks we need to complete so we can break it out into separate issues.

Tasks:

create an EventManager that processes and uploads events on a background thread - Create Android EventManager #2422
test low battery location strategies on Android
copy SSL certs to common folder then use build system to copy into ios and android

The text was updated successfully, but these errors were encountered:

bleege · 2015-11-12T01:23:42Z

The data model and lifecycle model to mirror from iOS can be found in MGLMapboxEvents.m.

bleege · 2015-11-12T01:33:58Z

Telemetry is going to need #2954 to be implemented first.

tobrun · 2016-01-11T15:27:53Z

I believe this is issue has gone out of date and is being tracked/evaluated elsewhere.

cc @bleege

bleege · 2016-01-11T15:30:59Z

Nope, we'll be using this ticket to build the implementation out. This will be easier to have data and history in one ticket.

…revent race condition issues

… to common location and updating iOS projects to load them from there

…ct for inclusion and use in SDK

…mpty json data to server

…e it after 24 hours

…rust

bleege · 2016-02-11T03:10:33Z

Rebased and merged into master so that internal testing can begin.

In order for Android Telemetry Iteration 1 to be considered "complete" the following tickets also must be completed:

Android Telemetry DigiCert Certificate Pinning #3888 - Digicert Pinning
Android Interaction Events #3889 - Interaction Events
Implement App Turnstile Event #3897 - Turnstile Event
Android Telemetry User Opt Out UI #3907 - Opt Out UI
Send Android Telemetry Events At App Close #4066 - Send Data At App Termination
Convert Android Telemetry To V2 Data Model #4071 - API V2
MapboxEventManager is using ApiAccess class #4077 - Remove ApiAccess Use

/cc @mapbox/telemetry

* master: (144 commits) [android] mapbox#3900 - Using CopyWriteArrayList to avoid concurrency issues with MapChangeListeners [android] mapbox#3919 - restore espresso tests [android] mapbox#3907 - Clearing any / all events in memory at the time of Telemetry being disabled [android] mapbox#3907 - Integrating Telemetry On / Off UI with functionality [android] mapbox#3907 - Initial refactoring to support on / off capability for Telemetry Services [android] mapbox#3907 - Adding Telemetry Settings to Attribution Options [tests] Fix build [core] Organize files that were in map folder better Allow using tileSize: 512 as a switch to trade retina support for 512px raster tiles [android] mapbox#3908 - No more beep [build] Build `offline` tool on the bots [build] Fix broken target when building on the bots [build] Build `offline` smoke test for Linux [android] mapbox#3876 - rename InfoWindowClickListener method [android] mapbox#3895 - handles the case of not requested/accepted permissions [core] use std::move() to prevent copies [core] return std::move() prevents copy elision [android] mapbox#2421 - Adding basic JavaDocs [android] mapbox#2421 - Setting up Certificate Pinning for Prod Server GeoTrust [android] mapbox#2421 - Setting up Certificate Pinning for Staging Server ...

afathman · 2016-03-27T23:56:59Z

My opinion on Telemetry: there should be an easy opt-out procedure, whether it be not including the service in the manifest or by using a MapboxMapOptions flag.

For me, using the Telemetry service in my manifest causes a crash in my application, and I'd also like to not send user location data as I'm just displaying a Map, not displaying the user's location.

bleege · 2016-03-28T15:47:53Z

Thanks for your feedback @afathman. We always love to hear from the community about Mapbox projects.

My opinion on Telemetry: there should be an easy opt-out procedure

We agree! That's why we built the Consumer Opt Out Control directly into the MapView UI so that all end users can control whether or not they want to participate.

For me, using the Telemetry service in my manifest causes a crash in my application

Can you tell us which version of the SDK you're using? Is it the latest Release Candidate from Friday night?

I'd also like to not send user location data as I'm just displaying a Map, not displaying the user's location.

The Telemetry system works by listening for Location updates from the app and operating system. The only time that User Location data will be generated by Mapbox tools is when the User Location is enabled using MapboxMap.setMyLocationEnabled(true).

afathman · 2016-03-28T15:53:27Z

We agree! That's why we built the Consumer Opt Out Control directly into the MapView UI so that all end users can control whether or not they want to participate.

Was hoping for a developer opt out control, it doesn't seem very friendly to crash an app when it doesn't list your Telemetry service in the manifest.

Can you tell us which version of the SDK you're using? Is it the latest Release Candidate from Friday night?

I'm using 4.0.0-rc1 in my build.

The Telemetry system works by listening for Location updates from the app and operating system. The only time that User Location data will be generated by Mapbox tools is when the User Location is enabled using MapboxMap.setMyLocationEnabled(true).

That makes sense... though I get logcat error messages from MapBox complaining that I didn't yet give it location permissions, even when I use setMyLocationEnabled(false);

bleege · 2016-03-28T16:21:01Z

I'm using 4.0.0-rc1 in my build.

Thanks for confirming. Can you provide a stack trace or more details about what's crashing?

though I get logcat error messages from MapBox complaining that I didn't yet give it location permissions, even when I use setMyLocationEnabled(false);

Yep, the Mapbox Android SDK uses the open source LOST framework which uses the stock AOSP Location API for Location data generation. This means that the usual Android Location permissions (ACCESS_COARSE_LOCATION and ACCESS_FINE_LOCATION) are required in the app.

ljbade added Android Mapbox Maps SDK for Android telemetry Integration with Mapbox Telemetry libraries labels Sep 25, 2015

ljbade added this to the android-v0.2.0 milestone Sep 25, 2015

ljbade mentioned this issue Sep 25, 2015

Create Android EventManager #2422

Closed

incanus removed this from the android-v0.2.0 milestone Sep 29, 2015

bleege added this to the android-v2.2.0 milestone Oct 26, 2015

ljbade added the P0 label Oct 27, 2015

bleege modified the milestones: android-v2.2.0, android-v2.3.0 Oct 28, 2015

bleege self-assigned this Nov 12, 2015

bleege changed the title ~~Android metrics planning~~ Android Telemetry planning Nov 12, 2015

bleege mentioned this issue Nov 12, 2015

Decouple MapView from LOST #2954

Closed

bleege modified the milestones: android-v2.3.0, android-v2.4.0 Dec 4, 2015

bleege modified the milestones: android-v3.0.0, android-v3.1.0 Dec 21, 2015

tobrun closed this as completed Jan 11, 2016

bleege reopened this Jan 11, 2016

bleege modified the milestones: android-v3.1.0, android-v4.0.0 Jan 20, 2016

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Bumping Session rotation to 24 hours

529896a

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Adding Location Permission checks to toggleGPS to p…

167cbec

…revent race condition issues

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Setting vendor id and session id to be encrypted

d88a350

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Adding WIFI_STATE to SDK permissions

0dfa2e5

bleege added a commit that referenced this issue Feb 11, 2016

[android][ios][core] - #2421 Moving Mapbox PROD and TEST certificates…

4719b73

… to common location and updating iOS projects to load them from there

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Copying PROD and Test certificates to Android proje…

0d639c7

…ct for inclusion and use in SDK

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Adding minimal events size check to avoid sending e…

5751302

…mpty json data to server

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Changing sessionId to UUID and adding flag to chang…

285cb33

…e it after 24 hours

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Setting up Certificate Pinning for Staging Server

adf83d3

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Setting up Certificate Pinning for Prod Server GeoT…

a945265

…rust

bleege added a commit that referenced this issue Feb 11, 2016

[android] #2421 - Adding basic JavaDocs

9709b3c

bleege closed this as completed Feb 11, 2016

jfirebaugh removed the in progress label Feb 11, 2016

This was referenced Feb 11, 2016

Implement App Turnstile Event #3897

Closed

Android Telemetry User Opt Out UI #3907

Closed

Remove Location Beep #3908

Closed

This was referenced Feb 13, 2016

Update OkHttp To 3.1.2 To Fix Security Vulnerability #3938

Closed

Clear Up Telemetry's encodeString #3960

Closed

This was referenced Feb 22, 2016

Send Android Telemetry Events At App Close #4066

Closed

Convert Android Telemetry To V2 Data Model #4071

Closed

This was referenced Mar 15, 2016

Release Android 4.0.0 Beta 1 #4317

Closed

Release Android 4.0.0 Beta 2 #4419

Closed

bleege mentioned this issue Mar 25, 2016

Release Android 4.0.0 Release Candidate 1 #4478

Closed

14 tasks

bleege mentioned this issue Mar 30, 2016

Release Android 4.0.0 Final #4535

Closed

13 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Android Telemetry planning #2421

Android Telemetry planning #2421

ljbade commented Sep 25, 2015

bleege commented Nov 12, 2015

bleege commented Nov 12, 2015

tobrun commented Jan 11, 2016

bleege commented Jan 11, 2016

bleege commented Feb 11, 2016

afathman commented Mar 27, 2016

bleege commented Mar 28, 2016

afathman commented Mar 28, 2016

bleege commented Mar 28, 2016

Android Telemetry planning #2421

Android Telemetry planning #2421

Comments

ljbade commented Sep 25, 2015

bleege commented Nov 12, 2015

bleege commented Nov 12, 2015

tobrun commented Jan 11, 2016

bleege commented Jan 11, 2016

bleege commented Feb 11, 2016

afathman commented Mar 27, 2016

bleege commented Mar 28, 2016

afathman commented Mar 28, 2016

bleege commented Mar 28, 2016