Closed Bug 1186080 Opened 9 years ago Closed 9 years ago

Consider using a hash of clientId to key Unified Telemetry data (instead of using clientId directly).

Categories

(Cloud Services Graveyard :: Metrics: Pipeline, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: mreid, Unassigned)

References

Details

(Whiteboard: [unifiedTelemetry])

In the interests of placing an upper bound on the length of the primary key used for longitudinal analysis, it may be better to use a hash of the raw clientId instead of the clientId itself. 

I filed this bug because I noticed a few crash submissions with the following:
"clientId":"593836                                                                             8c-ddc6-4cfa-91b2-24614767c823"

With all kinds of extra spaces.

Is the tradeoff in complexity and indirection worthwhile?
As discussed in today's triage, it would be better to validate that the client is sending well-formed UUIDs for the "clientId" field and then non-UUID values to the error stream on the server.
Priority: -- → P2
Whiteboard: [unifiedTelemetry]
See Also: → 1186488
Based on the discussion, should we close this as invalid?
Flags: needinfo?(mark.bugzilla)
Yep.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Flags: needinfo?(mark.bugzilla)
Product: Cloud Services → Cloud Services Graveyard
You need to log in before you can comment on or make changes to this bug.