browser not opening secure site

VERIFIED FIXED

Status

Core Graveyard
Security: UI
P1
critical
VERIFIED FIXED
17 years ago
a year ago

People

(Reporter: tracy, Assigned: Gagan)

Tracking

1.0 Branch

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

17 years ago
seen on commercial builds:

windows 2001-01-31-06-mtrunk
linux 2001-01-31-06-mtrunk

-attempt to go to the secure site above.

the browser does nothing.

Comment 1

17 years ago
Reassigning
Assignee: ddrinan → javi
Priority: -- → P1

Comment 2

17 years ago
psm starts up fine... and IMAP/S works, so this is a browser problem. gagan?

Comment 3

17 years ago
I just re-built on Mac and whenever I type https://, the following line asserts
http://lxr.mozilla.org/mozilla/source/netwerk/protocol/http/src/nsHTTPHandler.cpp#273

This looks like a necko problem.
Assignee: javi → gagan

Comment 4

17 years ago
adding self to cc list.

Comment 5

17 years ago
Created attachment 24019 [details] [diff] [review]
After quick code check, found a fix.
*** Bug 67249 has been marked as a duplicate of this bug. ***
I've got a slightly better fix, I think.  It lets us not put any HTTPS stuff in
nsHTTPHandler.  Patch coming up.
Created attachment 24049 [details] [diff] [review]
patch #2
r=dbaron, although perhaps this is a case where it would be better to skip the
optimization (maybe ask gagan once this is less urgent)?
a=ben@netscape.com
Checked in fix.  gagan, feel free to comment regarding what the "right fix" here is.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Assignee)

Comment 12

17 years ago
I completely agree with bryner here. we should keep HTTPS stuff out of HTTP as 
much as possible and his fix is the right one. 

Comment 13

17 years ago
Verified on the 2/1 Win, Mac and Linux builds.
Status: RESOLVED → VERIFIED

Comment 14

17 years ago
Mass changing Security:Crypto to PSM
Component: Security: Crypto → Client Library
Product: Browser → PSM
Version: other → 2.1

Comment 15

17 years ago
Mass changing Security:Crypto to PSM

Updated

13 years ago
Component: Security: UI → Security: UI
Product: PSM → Core

Updated

10 years ago
Version: psm2.1 → 1.0 Branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.