certutil is missing the description of the --email option

RESOLVED FIXED in 3.15.4

Status

NSS
Documentation
P2
normal
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: Elio Maldonado, Assigned: Elio Maldonado)

Tracking

3.15.1
3.15.4
x86_64
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

4 years ago
Created attachment 823528 [details] [diff] [review]
document certutil's --email option
(Assignee)

Updated

4 years ago
Assignee: nobody → emaldona
(Assignee)

Updated

4 years ago
Attachment #823528 - Attachment description: document certutil's ---email option → document certutil's --email option
(Assignee)

Updated

4 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

4 years ago
Attachment #823528 - Flags: review?(kaie)
(Assignee)

Comment 1

4 years ago
Running certutil -L --help
-L              List all certs, or print out a single named cert
   -n cert-name      Pretty print named cert (list all if unspecified)
   --email email-address 
                     Pretty print cert with email address (list all if unspecified)
   -d certdir        Cert database directory (default is ~/.netscape)
   -P dbprefix       Cert & Key database prefix
   -X                force the database to open R/W
   -r                For single cert, print binary DER encoding
   -a                For single cert, print ASCII encoding (RFC1113)
(Assignee)

Updated

4 years ago
Priority: -- → P2
Target Milestone: --- → 3.15.4

Comment 2

4 years ago
May I suggest a different wording, to make the purpose of the option more obvious?

The purpose wasn't immediately obvious to me. It's clear why you say "named certificate", since the email address is used for selecting the certificate. Since --email is an alternative for using the -n option, I propose a similar wording, based on the current wording for -n.

Proposed wording:
"Specify the email address of a certificate to list. Used with the -L command option."
(Assignee)

Comment 3

4 years ago
(In reply to Kai Engert (:kaie) from comment #2)
> Proposed wording:
> "Specify the email address of a certificate to list. Used with the -L command option."
Thank you, that's a lot clearer.
Summary: manpage for certutil is missing the description of the --email option → Specify the email address of a certificate to list. Used with the -L command option.
(Assignee)

Comment 4

4 years ago
Using a comma makes the title one sentence, a nitpick.
Summary: Specify the email address of a certificate to list. Used with the -L command option. → Specify the email address of a certificate to list, used with the -L command option.

Comment 5

4 years ago
Elio, my comment wasn't intended for the bug summary.

My suggestion is a proposal for a better wording for the contents in the manpage and in the patch!
(Assignee)

Updated

4 years ago
Summary: Specify the email address of a certificate to list, used with the -L command option. → certutil is missing the description of the --email option
(Assignee)

Comment 6

4 years ago
Created attachment 824923 [details] [diff] [review]
V2 - with Kai's clearer wording
Attachment #823528 - Attachment is obsolete: true
Attachment #823528 - Flags: review?(kaie)
Attachment #824923 - Flags: review?(kaie)

Comment 7

4 years ago
Comment on attachment 824923 [details] [diff] [review]
V2 - with Kai's clearer wording

Thanks!
r=kaie
Attachment #824923 - Flags: review?(kaie) → review+
(Assignee)

Updated

4 years ago
Target Milestone: 3.15.4 → 3.15.3
(Assignee)

Comment 8

4 years ago
Changes pushed to git https://hg.mozilla.org/projects/nss/rev/ad9713714ffe
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

4 years ago
s/git/mercurial/

Comment 10

4 years ago
You did not update the pre-built html/man-page documentation at the time of checkin.

This has been fixed with the checkin for bug 933688
https://hg.mozilla.org/projects/nss/rev/0560a4f7312a
(Assignee)

Comment 11

4 years ago
I forgot that part, will do it right now.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 12

4 years ago
Pushed the updates man pages and html pages
https://hg.mozilla.org/projects/nss/rev/4e3d88c7ffc3
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago4 years ago
Resolution: --- → FIXED

Comment 13

4 years ago
Elio, I said I already fixed it.

Comment 14

4 years ago
changing target milestone to 3.15.4
Target Milestone: 3.15.3 → 3.15.4
You need to log in before you can comment on or make changes to this bug.