Closed Bug 399698 Opened 15 years ago Closed 15 years ago

Cert Manager: also allow export of "exception certs" from the Servers tab

Categories

(Core :: Security: PSM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: mozbgz, Assigned: KaiE)

Details

Attachments

(1 file)

Attached patch Proposed fixSplinter Review
For certs which have been added with the "Add Exception..." button, cert manager currently disables the export, as it uses the "enable_edit" boolean to determine if its behavior (introduced with the changes from bug 327181).

I'm not sure if this was done on purpose (or just a copy-paste glitch), but I suggest to enable the export button whenever "View..." is available. Otherwise, the export button on the "Details" tab for this type of certificates would have to be disabled, too, if people should really be prevented from ever exporting such a cert.
Attachment #284752 - Flags: review?(kengert)
Comment on attachment 284752 [details] [diff] [review]
Proposed fix

r=kengert

Thanks Kaspar, this makes a lot of sense. This happened when I merged on check in.

Requesting approval.
Attachment #284752 - Flags: review?(kengert)
Attachment #284752 - Flags: review+
Attachment #284752 - Flags: approval1.9?
Attachment #284752 - Flags: approval1.9? → approval1.9+
fixed
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.