-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Fingerprinting 2.0: WebGL2 #9189
Comments
@pes10k @pilgrim-brave could one of you confirm the test plan? I don't see anything for |
Test plan is to review the "WebGL2 Params" row on https://dev-pages.brave.software/farbling.html and make sure they match on the same page, but differ across pages CC @pilgrim-brave @LaurenWags if that looks good to you, i think we're good to go! |
@pes10k thanks for getting that added! I do have a question about your setup on the test pages, looks like you've got 1.18 listed but this issue in 1.17.x: Can this issue be tested with 1.17.x or should I log a placeholder for it to be tested with 1.18.x? |
ah good point @LaurenWags , i tagged it as current nightly, not as the release when the code landed… d'oh. Fixed! |
Verified passed with
Verified test plan from #9189 (comment). Note, confirmed with @pes10k that when FP=strict, Verification passed on
Verified test plan from #9189 (comment). Verification passed on
Verified test plan from #9189 (comment). |
Verification passed on OnePlus 6T with Android 10 running 1.17.66 x64 build
Shields Down
Fingerprinting Blocked(Standard)
Fingerprinting Blocked(Strict)
Verification passed on Samsung Tab A with Android 10 running 1.17.66 x64 beta build Shields Down
Fingerprinting Blocked(Standard)
Fingerprinting Blocked(Strict)
|
This is a sub-issue of the larger fingerprint defense reorganization issue: #8787
Previous implementation blocked all WebGL-specific parameters, at max level only, and did nothing at default level.
New implementation:
At max level, block these WebGL2-specific parameters:
At default level, farble these WebGL2-specific parameters:
The text was updated successfully, but these errors were encountered: