Closed Bug 1580342 Opened 11 months ago Closed 10 months ago

Send the "sync ping" for all FxA users, not just Sync users

Categories

(Firefox :: Sync, enhancement, P2)

enhancement

Tracking

()

RESOLVED DUPLICATE of bug 1583897

People

(Reporter: markh, Unassigned)

References

(Blocks 1 open bug)

Details

Users will soon have the ability to be signed in to FxA but not to Sync. This means they will have the ability to send tabs - however, they will not be sending "sync pings". We should send "sync pings" for these users; even though there will be no actual syncs, it could still record other events, such as send tab.

This was done as part of bug 1583897

Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1583897

Is it accurate that we don't send this ping unless the user actually does some account-related action, such as sending a tab (and hence, that users who don't enable sync and don't use send-tab will not generate any sync pings in practice)?

Flags: needinfo?(markh)

(In reply to Ryan Kelly [:rfkelly] from comment #2)

Is it accurate that we don't send this ping unless the user actually does some account-related action, such as sending a tab (and hence, that users who don't enable sync and don't use send-tab will not generate any sync pings in practice)?

Yes, that's true - but I think that's OK, so maybe the title of this bug is just wrong. I guess it is analogous to the fact that we don't send a sync ping for sessions that don't sync. Note that SYNC_CONFIGURED and FXA_CONFIGURED are recorded in the main ping, so the sync ping isn't necessary to determine that state.

Lief, is the state described above OK from your perspective, or should we try and arrange to send a (basically) empty ping when no account/sync related events or actions happen?

Flags: needinfo?(markh) → needinfo?(loines)

I think its ok that we only send a ping when the user does something, so long as SYNC_CONFIGURED and FXA_CONFIGURED are reporting their values accurately. It will mean that people won't be able to use the sync telemetry to determine how many users have authenticated, but they probably shouldn't use it for that anyway.

Flags: needinfo?(loines)
You need to log in before you can comment on or make changes to this bug.