995452 - The FTU ping is successfully sent on the first attempt even if the Roaming SIM hasn't found the roaming network yet

Status: RESOLVED FIXED

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

Description

[Jason Smith [:jsmith]]

Build: 4/11/2014 1.5
Device: Buri

STR

1. Flash a build with a roaming SIM
2. Enable a wifi connection
3. Wait for the first attempt of the ping to be sent.

Expected

The ping should fail to be sent since the roaming network is not known.

Actual

The ping is successfully sent with the roaming network not known. Server-side data:

http://mockmetricsserver.appspot.com/6aec78ab-252a-4a98-8728-e17ccbaf5500/ftu/FirefoxOS/31.0a1/hamachi/1.5.0/nightly/20140411040203 
 {"ver":3,"screenHeight":480,"screenWidth":320,"devicePixelRatio":1,"locale":"en-US","pingID":"6aec78ab-252a-4a98-8728-e17ccbaf5500","activationTime":1397253399220,"deviceinfo.os":"1.5.0.0-prerelease","deviceinfo.software":"Boot2Gecko 1.5.0.0-prerelease","deviceinfo.platform_build_id":"20140411040203","deviceinfo.platform_version":"31.0a1","deviceinfo.product_model":"msm7627a","deviceinfo.firmware_revision":"","deviceinfo.hardware":"qcom","deviceinfo.update_channel":"hamachi/1.5.0/nightly","network":{"operator":null},"icc":{"mnc":"07","mcc":"214","spn":""},"pingTime":1397253430510}

Comment 1

[Jason Smith [:jsmith]]

Marshall - Any ideas?

Note - this will cause us to lose MCC/MNC data for SIMs that did not find their roaming network on a wifi connection.

Comment 2

[Jason Smith [:jsmith]]

Talked this over with Ravi - we can workaround this on the server side by determining what the country is on the server side via the wifi connection.

Comment 3

[Jason Smith [:jsmith]]

Mark - See comment 2. We're going to need to workaround this problem on the server-side for 1.3 by using the IP address of the wifi connection to determine the country in this bug's use case. Does that make sense?

Comment 4

[Jason Smith [:jsmith]]

Looks like Ravi already talked to John Jensen about this.

Comment 5

[Mark Reid [:mreid]]

Currently the client IP address is discarded as early as possible in the Telemetry server code. I'll have to modify the code to hang on to it a little longer so we can do the IP -> Country lookup.

Comment 6

[Mark Reid [:mreid]]

I've deployed new code to temporarily store the IP address, allowing us to do GeoIP lookups for the FTU pings. Note that the IP address itself is still never stored long-term and is discarded after processing.

IP changes landed in:
https://github.com/mozilla/telemetry-server/pull/48

Geocoding landed in:
https://github.com/mozilla/telemetry-server/pull/49

More details in Bug 997960.