-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
The stats API allow hardware fingerprinting (encoder, powerEfficient) #675
Comments
Relying on context-capture-state is not perfect but good enough to take us forward, and I like the phrase leaving the door open for other "deep user interactions". It resolves the immediate fingerprinting issue, though it does prevent exposure in some previously available use cases. But if we have a good privacy baseline we could iterate and add other exposure triggers in the future if that becomes necessary. |
@pes10k thoughts? |
I think the general approach is appealing, but i dont think "in a deep manner" is specific enough guidance for implementors. It would be ideal to just comprehensively mention when the field should be available and when it shouldn't |
This particular vagueness was phrased by @youennf. Youenn what if we add a hardware mitigation algorithm section that we refer to, and for now we make it explicitly that you MUST have "context capture state = true". While this does not have any vagueness, we would have a go-to place in the spec if we want to add other steps in the future where it should also be allowed to expose a metric. It would be nice if we could be in a state where we're over-protective instead of under-protective allowing more discussion when the time comes that we want to expose the metrics to more use cases. I feel like we get stuck on trying to have something be perfect and then we don't progress. Better to be over-protective and revisit, I say, so we can unblock the PR. |
This sounds good to me. |
Done: #670 |
From #550:
For the
codec
issue, I filed #674 as a separate issue due to the tie with webrtc-pc.In the most recent TPAC (yesterday) we talked about fingerprinting in the context of adding
powerEfficientEncoder
.@youennf added to #550 after the meeting:
I like this way of phrasing it and think we should add this to
encoderImplementation
anddecoderImplementation
, as well as the recently proposedpowerEfficientEncoder
andpowerEfficientDecoder
(#666).The text was updated successfully, but these errors were encountered: