Opened 6 years ago
Closed 6 years ago
#14203 closed defect (fixed)
Tor Browser with meek opens two Software Update windows
| Reported by: | dcf | Owned by: | dcf |
|---|---|---|---|
| Priority: | Medium | Milestone: | |
| Component: | Circumvention/meek | Version: | |
| Severity: | Keywords: | TorBrowserTeam201502 | |
| Cc: | brade, mcs | Actual Points: | |
| Parent ID: | Points: | ||
| Reviewer: | Sponsor: |
Description
When I'm browsing with meek, I tend to get two "Software Update Available" windows appearing simultaneously. I suppose the second one is from the headless meek-http-helper.
Child Tickets
Attachments (1)
Change History (9)
comment:1 Changed 6 years ago by
Changed 6 years ago by
| Attachment: | 0001-Disable-software-update-windows-in-the-meek-HTTP-hel.patch added |
|---|
comment:2 Changed 6 years ago by
| Keywords: | TorBrowserTeam201502 added |
|---|---|
| Status: | new → needs_review |
attachment:0001-Disable-software-update-windows-in-the-meek-HTTP-hel.patch sets app.update.enabled=false. I tested it by manually setting the setting in the user.js of 4.0.2, and seeing that only one software update window appeared.
comment:3 follow-up: 5 Changed 6 years ago by
| Cc: | brade mcs added |
|---|
The patch looks good. Two questions:
- Do you have commit access to builders/tor-browser-bundle.git ?
- Do you we want this fix in 4.0.x or just in 4.5 ? (or maybe I should ask mikeperry).
comment:4 Changed 6 years ago by
| Keywords: | TorBrowserTeam201502R added; TorBrowserTeam201502 removed |
|---|
comment:5 Changed 6 years ago by
Replying to mcs:
The patch looks good. Two questions:
- Do you have commit access to builders/tor-browser-bundle.git ?
No, but I kind of prefer not to have. I'm fine with asking for patches to be merged.
- Do you we want this fix in 4.0.x or just in 4.5 ? (or maybe I should ask mikeperry).
I don't care. I'm not aware of any user complaints about the two update windows, so it's fine if you want to try it in an alpha first.
comment:6 follow-up: 7 Changed 6 years ago by
I am concerned that accepting the meek version of the update notice will cause surprise issues, perhaps even leading to a damaged TBB directory. This is making me think we should take this change for both the stable and the alpha TBB.
comment:7 Changed 6 years ago by
Replying to mikeperry:
I am concerned that accepting the meek version of the update notice will cause surprise issues, perhaps even leading to a damaged TBB directory. This is making me think we should take this change for both the stable and the alpha TBB.
Good point. At best, untested. At worst, completely broken. Certainly confusing. So I agree with you.
comment:8 Changed 6 years ago by
| Keywords: | TorBrowserTeam201502 added; TorBrowserTeam201502R removed |
|---|---|
| Resolution: | → fixed |
| Status: | needs_review → closed |
Ok, pushed to maint-4.0 and master, for 4.0 and 4.5. Thanks, David!
It looks like we can turn off updating in the headless browser by setting the pref app.update.enabled=false.
https://developer.mozilla.org/en-US/docs/Mozilla/Projects/XULRunner/Application_Update
I'll try that and see if it works.