Open Bug 1651387 Opened 5 years ago

Hide type IPCPayload to discourage external uses of FOG IPC

Categories

(Toolkit :: Telemetry, task, P3)

Product:
Toolkit
Component:
Telemetry
Type:
task

Tracking

()

People

(Reporter: chutten, Unassigned)

Details

(Whiteboard: [telemetry:fog:m?])

struct IPCPayload is currently publicly accessible on the glean crate. (and we use that fact in tests)

If we can hide that type then we will make it exceedingly difficult for anyone to misuse the FOG IPC mechanism by crafting their own payloads.

On the other hand, we haven't had any problems with people misusing Telemetry IPC with its copious quantity of public structs, so maybe we shouldn't bother at all? Especially if it makes testing difficult?

We should weight the cost/benefit of this, at any rate.

You need to log in before you can comment on or make changes to this bug.