All users were logged out of Bugzilla on October 13th, 2018
This menu is not relevant unless we support any more options other than the 'update from last fetched url'
Assignee: ssaux → rangansen
Status: ASSIGNED → NEW
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.
Kai, Please review ...
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?
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 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 on attachment 61138 [details] [diff] [review] patch sr=hewitt
Attachment #61138 - Flags: superreview+
Patch checked in ...
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.