Closed Bug 1426602 Opened 3 years ago Closed 3 years ago

Remove certcgi command

Categories

(NSS :: Test, enhancement)

3.34
enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: franziskus, Assigned: jkt)

References

Details

Attachments

(1 file)

The certcgi "command" is a test that doesn't seem to be used. It should probably be removed.
Duplicate of this bug: 1426672
Assignee: nobody → jkt
Comment on attachment 8938380 [details] [diff] [review]
bug-1426602.patch

Review of attachment 8938380 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm
But I'll wait with landing for the new release (a week or so).

Kai, do you know of any users of this? The keygen tag (needed in certcgi) is only available in Firefox and will probably be removed in 61. So I think we should drop this command unless there's a significant user base.
Attachment #8938380 - Flags: superreview?(kaie)
Attachment #8938380 - Flags: review?(franziskuskiefer)
Attachment #8938380 - Flags: review+
Target Milestone: --- → 3.36
Comment on attachment 8938380 [details] [diff] [review]
bug-1426602.patch

We aren't aware of users of certcgi. Removal should be OK.
Attachment #8938380 - Flags: superreview?(kaie) → superreview+
https://hg.mozilla.org/projects/nss/rev/6c058f8139640e7e150e79073ebcc815a8aeaf79
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.