Skip to content
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

FPP canvas can fail in service workers [1878716] #1813

Open
Thorin-Oakenpants opened this issue Feb 28, 2024 · 12 comments
Open

FPP canvas can fail in service workers [1878716] #1813

Thorin-Oakenpants opened this issue Feb 28, 2024 · 12 comments

Comments

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented Feb 28, 2024


Originally posted by @mik0l in #1716 (comment)

Not always, but sometimes there is a leak in "Service".

  1. Start browser and paste into the address bar https://abrahamjuliot.github.io/creepjs/tests/workers.html
  2. If there is no leak, close browser and return to step 1.

Originally posted by @Thorin-Oakenpants in #1716 (comment)

OK, so I can confirm (nightly 125) - the result in SWers sometimes (I get it almost every time) is not randomized

@tomrittervg see STR ~~in previous comment

this is without FPP
noFPP

some FPP examples

diff1

diff2

@Thorin-Oakenpants
Copy link
Contributor Author

Thorin-Oakenpants commented Feb 28, 2024

spinning this off to a separate issue so I can track it

@Thorin-Oakenpants
Copy link
Contributor Author

^ ping! @tomrittervg reminder .. next reminder in 3 more days :)

@tomrittervg
Copy link

I have it in my list, I'm in training the next couple days, so I'm not sure if I'll get to it before next reminder ;)

@Thorin-Oakenpants
Copy link
Contributor Author

no worries, I'll stop pinging you now :) But this is a blocker for #1804 and unlocking the hundreds of millions of arkenfox users into FPP for testing and reporting 👀

@Thorin-Oakenpants
Copy link
Contributor Author

@Thorin-Oakenpants Thorin-Oakenpants changed the title FPP canvas can fail in service workers FPP canvas can fail in service workers [1885471] Mar 15, 2024
@tomrittervg
Copy link

It looks like the window.open patches in https://bugzilla.mozilla.org/show_bug.cgi?id=1878716 will resolve this also

@Thorin-Oakenpants
Copy link
Contributor Author

Thorin-Oakenpants commented Apr 5, 2024

@Thorin-Oakenpants Thorin-Oakenpants changed the title FPP canvas can fail in service workers [1885471] FPP canvas can fail in service workers [1878716] Apr 6, 2024
@Thorin-Oakenpants
Copy link
Contributor Author

Thorin-Oakenpants commented May 2, 2024

@tomrittervg did you know (at least FF125+) that if you have RFP enabled and you have ETP Strict enabled - that when you relax RFP canvas extraction, FPP kicks in

here's an example showing RFP is on with RFP values for speech engines, media devices, audio context keys + values, etc .. with said conditions above (canvas exception, ETP blue shield .. it's strict)

tada

I wonder if this would happen in TB next ESR since it's already in PB mode (hence FPP by default) - cc @pospeselr

edit: so RFP overrides FPP where RFP is used. But if FPP adds protections not covered by RFP then there's nothing to stop them from being applied - e.g. when RFP canvas is excepted. This is probably fine, since anything FPP adds can only be a net gain

@tomrittervg
Copy link

Yeah... I think we should have a bug on file for this. Canvas is just unusual because there's 4 behaviors to account for across the 2 different modes and then 2 ways of exempting a site that affect the modes differently.

@Thorin-Oakenpants Thorin-Oakenpants changed the title FPP canvas can fail in service workers [1878716] FPP canvas can fail in service workers [1878716 fixed 127] May 3, 2024
@Thorin-Oakenpants Thorin-Oakenpants changed the title FPP canvas can fail in service workers [1878716 fixed 127] FPP canvas can fail in service workers [1878716] May 3, 2024
@Thorin-Oakenpants
Copy link
Contributor Author

Thorin-Oakenpants commented May 3, 2024

gah .. still failing: STR

  • turn off ETP Strict and use non-PB mode (and disable RFP)
  • new session load https://abrahamjuliot.github.io/creepjs/tests/workers.html
  • get your real canvas value: for me that is 0e8c9761 (on all four tests)
  • turn on ETP Strict
  • close browser (and I sanitize everything on close)
  • test
  • close, open
  • test
  • rinse repeat

It's not applying any FPP on canvas

@Thorin-Oakenpants
Copy link
Contributor Author

Thorin-Oakenpants commented May 3, 2024

weird - TZP shows FPP is used .. but that creepy test is showing no changes - it always shows 0e8c9761 which is my real value

edit: even weirder : PB mode is not showing FPP canvas - i,e we are returning the expected values as per TZP test I'm an idiot, accidentally opened file:// instead of https

@Thorin-Oakenpants
Copy link
Contributor Author

Yeah... I think we should have a bug on file for this. Canvas is just unusual because there's 4 behaviors to account for across the 2 different modes and then 2 ways of exempting a site that affect the modes differently.

https://gitlab.torproject.org/tpo/applications/tor-browser/-/issues/42556

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants