Opened 5 years ago

Last modified 2 weeks ago

#13496 assigned task

Write test for spoofed navigator* values in workers

Reported by: gk Owned by: tbb-team
Priority: Medium Milestone:
Component: Applications/Tor Browser Version:
Severity: Normal Keywords: ff38-esr, tbb-testcase
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

#13027 fixed the problem with navigator* values not being spoofed. Back then we could not write a Mozmill test as Mozmill itself was affected by the underlying issue (https://bugzilla.mozilla.org/show_bug.cgi?id=1062920 together with https://bugzilla.mozilla.org/show_bug.cgi?id=1078163).

Child Tickets

Change History (5)

comment:1 Changed 5 years ago by gk

Keywords: tbb-testcase added

A preliminary test is in bug_13496 in my public test repo. Reminder: don't forget to test nested workers.

comment:2 Changed 2 years ago by teor

Severity: Normal

Set all open tickets without a severity to "Normal"

comment:3 in reply to:  1 Changed 4 weeks ago by Thorin

Replying to gk:

A preliminary test is in bug_13496 in my public test repo. Reminder: don't forget to test nested workers.

I think we can close this. Override prefs are not used anymore see tom's comment - https://trac.torproject.org/projects/tor/ticket/27268#comment:18 "Yeah all of these should be deleted from tor's js file. With RFP enabled they do nothing"

---

PS: FYI: I know this is not what the ticket was for, but you can go to https://securehomes.esat.kuleuven.be/~gacar/dev/worker_test.html which shows web workers return RFP values (although some spoofed values are the real values, e.g Win32 on windows machines) - I might add this to TZP but it's probably a waste of time

PPS: do you want me to test RFP UA spoofing in nested WebWorkers? Or would that also be a waste of time - anyway, feel free to close this

comment:4 Changed 4 weeks ago by gk

I think we should make a decision about closing that one once we move to a non-ESR Firefox and are sure the regular Firefox tests available for spoofing are enough. If not, we can use this ticket to write missing ones, otherwise close it.

Thorin: It sounds there might be tests missing (reading you PPS)? If so, feel free to go ahead and check it out.

comment:5 Changed 2 weeks ago by gk

Owner: changed from gk to tbb-team
Status: newassigned

re-assigning

Note: See TracTickets for help on using tickets.