Closed
Bug 431940
Opened 17 years ago
Closed 15 years ago
Port |Bug 402207 – SSL error page "Add Exception" support should be pref-controlled| to SeaMonkey
Categories
(SeaMonkey :: Security, defect)
SeaMonkey
Security
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.0b2
People
(Reporter: sgautherie, Assigned: sgautherie)
References
Details
Attachments
(1 file, 1 obsolete file)
|
(deleted),
patch
|
iannbugzilla
:
review+
|
Details | Diff | Splinter Review |
SeaMonkey misses the preference handling from the two </browser/*> files.
The current behavior seems to be |value = 1|,
but is not configurable.
|
Assignee | |
Comment 1•17 years ago
|
||
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008050202 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)
|
|
Assignee | |
Updated•17 years ago
|
Attachment #319100 -
Flags: review? → review?(iann_bugzilla)
Attachment #319100 -
Flags: review?(iann_bugzilla) → review+
|
|
Assignee | |
Updated•17 years ago
|
Keywords: checkin-needed
|
||
Comment 2•17 years ago
|
||
Does this need SR? I'm not 100% sure about SeaMonkey review rules.
|
|
||
Comment 3•17 years ago
|
||
Comment on attachment 319100 [details] [diff] [review]
(Av1) </suite/*>
Not sure if this needs SR or not. Please advise.
Attachment #319100 -
Flags: superreview?(neil)
|
|
Assignee | |
Comment 4•16 years ago
|
||
Neil, ping ?
|
||
Comment 5•16 years ago
|
||
Removing for now as there seems to be a pending sr requirement.
Keywords: checkin-needed
|
|
Assignee | |
Updated•15 years ago
|
|
|
Assignee | |
Comment 6•15 years ago
|
||
Av1, updated to current trunk.
(In reply to comment #3)
> Not sure if this needs SR or not. Please advise.
/suite/ only patches don't (nowadays).
Attachment #319100 -
Attachment is obsolete: true
Attachment #392806 -
Flags: review?(iann_bugzilla)
Attachment #319100 -
Flags: superreview?(neil)
Comment on attachment 392806 [details] [diff] [review]
(Av1a) Port bug 402207 + bug 407369 + bug 431826 (relevant part)
[Checkin: Comment 9]
r=me
Attachment #392806 -
Flags: review?(iann_bugzilla) → review+
|
||
Comment 8•15 years ago
|
||
(In reply to comment #4)
> Neil, ping ?
Sorry, I get so many requests that from time to time some sink without trace :-(
Regarding the patch, I wonder why Firefox bothered with the try/catch...
|
|
Assignee | |
Comment 9•15 years ago
|
||
Comment on attachment 392806 [details] [diff] [review]
(Av1a) Port bug 402207 + bug 407369 + bug 431826 (relevant part)
[Checkin: Comment 9]
http://hg.mozilla.org/comm-central/rev/4344cc9a28a7
(In reply to comment #8)
> Regarding the patch, I wonder why Firefox bothered with the try/catch...
Bug 402207 doesn't say.
I assume they prefer to fall back to the default behavior rather than throwing, in case anything/whatever goes wrong with the pref retrieval.
Yet, feel free to file a bug to get try+catch removed from FF (and SM), if need be.
Attachment #392806 -
Attachment description: (Av1a) Port bug 402207 + bug 407369 + bug 431826 (relevant part) → (Av1a) Port bug 402207 + bug 407369 + bug 431826 (relevant part)
[Checkin: Comment 9]
|
|
Assignee | |
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•