Closed
Bug 112208
Opened 23 years ago
Closed 20 years ago
Autocomplete doesn't match against redirected urls
Categories
(Core Graveyard :: History: Global, defect)
Core Graveyard
History: Global
Tracking
(Not tracked)
VERIFIED
WORKSFORME
Future
People
(Reporter: markus.langstrom, Assigned: mikepinkerton)
References
()
Details
(Keywords: regression, top100, Whiteboard: [adt3])
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
Details | Diff | Splinter Review |
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.6+) Gecko/20011126
BuildID: 2001112603
Can't find any dups of this...
Reproducible: Always
Steps to Reproduce:
1. Enable Location Bar Autocomplete
2. Visit http://mail.yahoo.se/
3. Type mail.. in location bar.
Actual Results: Autocomplete doesn't work
Expected Results: The URL should be "autocompleted"
History->Global. The link you provided redirects to
http://login.europe.yahoo.com/config/mail?.intl=se whirch is saved in location
bar history, instead of mail.yahoo.se. Hense, only by typing "login"
autocompletes the target url, as expected. Mozilla doesn't keep in either url
bar or global history the originating urls (those redirecting to another
location). Suggesting wontfix.
Assignee: hewitt → radha
Status: UNCONFIRMED → NEW
Component: XP Apps: Autocomplete → History: Session
Ever confirmed: true
QA Contact: blakeross → claudius
Comment 2•23 years ago
|
||
urlbar autocomplete is not a feature od session history. It is associated to
global history.
Assignee: radha → alecf
Component: History: Session → History: Global
Reporter | ||
Comment 4•23 years ago
|
||
"History->Global. The link you provided redirects to
http://login.europe.yahoo.com/config/mail?.intl=se whirch is saved in location
bar history, instead of mail.yahoo.se. Hense, only by typing "login"
autocompletes the target url, as expected. Mozilla doesn't keep in either url
bar or global history the originating urls (those redirecting to another
location). Suggesting wontfix."
But it worked before (a few days/weeks ago)...
FWIW, I still think it should be fixed ;-)
Comment 5•23 years ago
|
||
Also seeing this on slashdot.org (with 200112903, used to work in older builds).
Entering "www.sla" would autocomplete; now, since the URL is actually
"http://slashdot.org", mozilla will not complete "www.sla" to "www.slashdot.org".
Suggesting higher severity (normal), as *MANY* users will be confused by this
behaviour.
Reporter | ||
Comment 6•23 years ago
|
||
Even worse, Hotmail shows the same behavior!
This will indeed confuse MANY users...
Changing severity to Normal.
Severity: minor → normal
Comment 7•23 years ago
|
||
This is not an autocomplete bug, history should be storing these urls but it is not
Assignee: hewitt → blakeross
Summary: Autocomplete doesn't work with this URL → history doesn't store urls redirected urls
Summary: history doesn't store urls redirected urls → history doesn't store redirected urls
*** Bug 116729 has been marked as a duplicate of this bug. ***
Updated•23 years ago
|
Target Milestone: --- → Future
Not platform specific.
Maybe dependent on bug 48202 but not sure.
Severity: normal → major
OS: Windows 98 → All
Hardware: PC → All
Comment 10•23 years ago
|
||
We do store these redirect urls but they're just hidden in the UI, right? We
need to ensure that autocomplete still matches against them.
Comment 11•23 years ago
|
||
Comment 12•23 years ago
|
||
alec/hewitt: I think we should still match against hidden urls even if we don't
show them in the history window. IE hides redirected urls just like us but
their autocomplete still matches against them. It would be a shame to make
users who check their e-mail every day type `lc2.law13' to autocomplete hotmail.com.
Updated•23 years ago
|
Summary: history doesn't store redirected urls → Autocomplete doesn't match against redirected urls
Comment 13•23 years ago
|
||
Comment on attachment 68735 [details] [diff] [review]
patch?
no! if we do this then mozilla will be blocked from MSN, among other sites.
Here's what we should be doing:
- add a new mork column, "typed" (like "hidden") which determines if the row
was entered on the URL bar.
- add an API to nsIBrowserHistory to flag a url as "typed"
- when a user hits enter in the URL bar, turn the text entered into a url, and
then call that nsIBrowserHistory option
- fix the autocomplete enumerator to check if hidden urls are also "typed" - if
so, then return them.
Attachment #68735 -
Flags: needs-work+
Comment 14•23 years ago
|
||
I should add that I really am excited about getting this bug fixed - another
common one is amazon.com - you'll note that no matter how many times you visit
amazon, "http://www.amazon.com/" is never in the autocomplete.
Comment 15•23 years ago
|
||
Umm...okay...but what does this mean:
> no! if we do this then mozilla will be blocked from MSN, among other sites.
Why does this make us be blocked?
Comment 16•23 years ago
|
||
because 304 redirects are sometimes used to obscure hidden information like
passwords, and such. We cannot ever display the results of 304 redirect to users
on screen. Yes, I know they are stored unencrypted on the filesystem, but
certain sites (Such as MSN) will block clients that are known to display 304
redirected urls on the screen. This is why to this day Netscape 6.0-6.2 is
blocked from MSN.
Comment 17•23 years ago
|
||
I think you don't mean 304 Not modified response... (Perhaps 302, 303 or 307.)
I think that MSN is really really broken if they send passwords in cleartext in
GET requests. They shouldn't do that and they shouldn't block browsers that
autocomplete to original adresses of temporary redirects.
So they should use POST method to send passwords (I don't say that this is
perfectly secure :-)) and generate 303 response on that request if they want.
But the browser should autocomplete to the ORIGINAL address. I wouldn't mind if
I couldn't rather autocomplete to the destination address than to the original
address (esp. on 307 response).
Keywords: mozilla1.0
Comment 18•23 years ago
|
||
yes, we KNOW that, but all other browsers hide 304 redirects (or whatever, I was
told they were 304's, I'm just interpreting a REDIRECT flag that comes in from
necko) from global history, and I believe it is the right thing to do, because I
think there are lots of legitimate uses of 3xx redirects that would otherwise
pollute the global history UI with all sorts of garbage.
What I'm suggesting doing is overriding the hide-the-304 behavior, but ONLY when
a user has actually typed the url at the url bar.
Comment 19•23 years ago
|
||
I agree with alec, and on closer inspection, this is what IE does (only if I
type in hotmail.com will it autocomplete it in the urlbar).
Comment 20•23 years ago
|
||
Attachment #68735 -
Attachment is obsolete: true
Comment 21•23 years ago
|
||
*** Bug 125281 has been marked as a duplicate of this bug. ***
Comment 22•23 years ago
|
||
Cc'ing alec for sr (well, not really. but since he lost his mail, here's a
reminder ;)
Comment 23•23 years ago
|
||
Comment on attachment 69156 [details] [diff] [review]
patch
cool, so the fixupURI thing will turn "amazon.com" into "http://amazon.com" or
something similar? (just to confirm)
sr=alecf
Attachment #69156 -
Flags: superreview+
Comment 24•23 years ago
|
||
Comment on attachment 69156 [details] [diff] [review]
patch
r=hewitt
Attachment #69156 -
Flags: review+
Comment 26•23 years ago
|
||
*** Bug 126728 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 27•23 years ago
|
||
reopening. as i noticed in bug 162325, we are not purging sites at all, so the
pre-redirect site will not get purged and will appear to work. This purging
should happen in nsGlobalHistory::MatchExpiration() at shutdown.
on the branch, redirected sites are purged so they don't show up in autocomplete
(chimera bug 159013).
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 28•22 years ago
|
||
this patch checks the "num visited" column when purging a typed and hidden url
from the history. if it has been visited and it's typed and hidden, it's a
redirect and we should keep it. if it hasn't been visited, it's an invalid
typed site.
looking for r/sr for trunk. i've already landed this on the chimera branch.
Attachment #69156 -
Attachment is obsolete: true
Comment 29•22 years ago
|
||
Nav triage team: nsbeta1+/adt3
Comment 30•22 years ago
|
||
Mike,
Would you be able to drive your patch to checkin?
Assignee: blaker → pinkerton
Status: REOPENED → NEW
Comment 31•22 years ago
|
||
*** Bug 200175 has been marked as a duplicate of this bug. ***
Comment 32•22 years ago
|
||
this is broken again. http://mail.yahoo.com and http://www.espn.com don't work
as of build 2003051908, on lunix or windows. please, please fix.
Comment 33•22 years ago
|
||
please, please, offer up a patch.
Comment 34•22 years ago
|
||
*** Bug 186531 has been marked as a duplicate of this bug. ***
Comment 35•21 years ago
|
||
*** Bug 216441 has been marked as a duplicate of this bug. ***
Comment 38•21 years ago
|
||
*** Bug 224533 has been marked as a duplicate of this bug. ***
Comment 39•21 years ago
|
||
*** Bug 222795 has been marked as a duplicate of this bug. ***
Comment 40•21 years ago
|
||
*** Bug 244483 has been marked as a duplicate of this bug. ***
Comment 41•21 years ago
|
||
*** Bug 233959 has been marked as a duplicate of this bug. ***
Comment 42•20 years ago
|
||
This WORKSFORME with linux trunk 2005042102. Is anyone still seeing this bug?
Comment 43•20 years ago
|
||
You're right, this WFM as well using a 20050417 Windows Firefox build. Resolving
worksforme, please reopen or comment if someone can find a way to reproduce.
Status: NEW → RESOLVED
Closed: 23 years ago → 20 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Updated•15 years ago
|
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•