All users were logged out of Bugzilla on October 13th, 2018

IN CRL autoupdate prefs window, should remove the menu for update from url

VERIFIED FIXED in psm2.2

Status

P1
normal
VERIFIED FIXED
17 years ago
2 years ago

People

(Reporter: rangansen, Assigned: rangansen)

Tracking

1.0 Branch
psm2.2
All
Windows NT

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

17 years ago
This menu is not relevant unless we support any more options other than the 
'update from last fetched url'
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 1

17 years ago
->me
Assignee: ssaux → rangansen
Status: ASSIGNED → NEW

Updated

17 years ago
Priority: -- → P1
Target Milestone: --- → 2.2
(Assignee)

Comment 2

17 years ago
Created attachment 61138 [details] [diff] [review]
patch

Removed the idea of advertised url altogether - untill we have support for the
same in NSS. So, urlType also goes away from prefs. Its always last fetch url.
(Assignee)

Comment 3

17 years ago
Kai, Please review ...

Comment 4

17 years ago
Patch looks good, but one question: Why do you store the preference, when there
is only one possible setting? Should we remove that code, too?
(Assignee)

Comment 5

17 years ago
Do you mean the storing the url string? [The url type is not being stored 
any amore]Because the autoupdate scheduler 
[nsNSSComponent::getParamsForNextCrlToDownload ]checks for which crl to update 
next, and then tries to import the crl directly from the corresp stored url 
string. This operation is much less costly than looking up for the crl in the 
db, and and finding out the lastfetchurl String 

Comment 6

17 years ago
Comment on attachment 61138 [details] [diff] [review]
patch

I thought you were storing a flag whether you do "last fetch" or "as
advertised".

r=kaie
Attachment #61138 - Flags: review+

Comment 7

17 years ago
Comment on attachment 61138 [details] [diff] [review]
patch

sr=hewitt
Attachment #61138 - Flags: superreview+
(Assignee)

Comment 8

17 years ago
Patch checked in ...
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 9

17 years ago
Verified.
Status: RESOLVED → VERIFIED

Updated

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

Updated

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