Closed
Bug 344029
Opened 18 years ago
Closed 18 years ago
going back to home page in antiphishing breaks if multiple tab homepage is set
Categories
(Toolkit :: Safe Browsing, defect, P1)
Tracking
()
RESOLVED
FIXED
Firefox 2
People
(Reporter: chaseslabaugh, Assigned: Gavin)
References
Details
(Keywords: fixed1.8.1, Whiteboard: [patch in 339032])
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060707 Firefox/2.0b1 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060707 Firefox/2.0b1 If you go to a site that is suspected for phishing, and then click Get me out of here! to leave, if your homepage contains tabs, such as mine (http://slashdot.org/|http://digg.com/), then it will attempt to navigate to that site, i.e. going to the nonexistent slashdot directory |http://digg.com, instead of opening 2 tabs. Reproducible: Always Steps to Reproduce: 1. Make sure homepage is multi tab, like http://slashdot.org/|http://digg.com/ 2. Go to suspected phishing site, such as Google's test site. 3. Click the Get me out of here! option. Actual Results: It attempted to navigate to http://slashdot.org/|http://digg.com/ NOT to slashdot and digg in 2 separate tabs like they should. Expected Results: Opened 2 tabs, one with Digg, and one with Slashdot.
Comment 2•18 years ago
|
||
I will confirm this *for now*, as I can reproduce this in the latest Bon Echo nightly (Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1b1) Gecko/20060810 BonEcho/2.0b1). But yes, I'd love to see "get me out of here" do something more relevant.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•18 years ago
|
Target Milestone: --- → Firefox 2
Updated•18 years ago
|
Flags: blocking-firefox2?
Target Milestone: Firefox 2 → Firefox 2 beta2
Updated•18 years ago
|
Flags: blocking-firefox2? → blocking-firefox2+
Updated•18 years ago
|
Target Milestone: Firefox 2 beta2 → Firefox 2
Comment 3•18 years ago
|
||
So, window.home() breaks pretty badly if there's multiple homepages, we should a) decide what it should do and then b) make that happen options are: 1) open first page 2) open all pages 3) fail brutally and shrug our shoulders (3 really isn't an option)
Assignee: nobody → gavin.sharp
Comment 4•18 years ago
|
||
or 4) Open the home page if it's one page, or fail gently if it's not. Looks to me like it's not calling window.home(), it's getting a URL in getMeOutOfHereUrl_() which defaults to about:blank, then tries to get the home page pref, and returns that (to loadURI()) if it can. The least painful non-scary thing would seem to me to .test() the pref value, and return about:blank if there's a pipe in it. In my multiple-home-page setups, having only the first page load would be more puzzling than comforting, and having all of them pop up would be more annoying than comforting.
Assignee | ||
Comment 5•18 years ago
|
||
(In reply to comment #3) > So, window.home() breaks pretty badly if there's multiple homepages, we should > a) decide what it should do and then b) make that happen As Phil said, fixing window.home() to handle multiple home pages wouldn't fix this bug, since it's not used to load the home page. That problem is already filed as bug 343999. Given that Firefox is one of the only gecko apps to have "|" separated home pages, achieving adequate, non-hacky behavior is not trivial. How about I fix bug 339032 instead? It should be fairly trivial, assuming beltzner's 339032 comment 14 describes the behavior we want.
Status: NEW → ASSIGNED
OS: Windows XP → All
Priority: -- → P1
Hardware: PC → All
Version: unspecified → 2.0 Branch
Assignee | ||
Updated•18 years ago
|
Whiteboard: [patch in 339032]
Assignee | ||
Comment 6•18 years ago
|
||
Fixed by the checkin for bug 339032.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Firefox → Toolkit
You need to log in
before you can comment on or make changes to this bug.
Description
•