Closed
Bug 168976
Opened 22 years ago
Closed 22 years ago
Remove RsaPubModulus from nsIX509Cert
Categories
(Core Graveyard :: Security: UI, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: KaiE, Assigned: KaiE)
References
Details
Attachments
(1 file)
|
2.65 KB,
patch
|
javi
:
review+
rpotts
:
superreview+
|
Details | Diff | Splinter Review |
I'm in the process of adding documentation to nsIX509Cert. I see it contains a readonly attribute named rsaPubModulus. Our implementation always returns the string "not yet implemented". None of our code actually accesses that attribute. I propose to remove this completely from the interface and the implementation.
|
||
Comment 1•22 years ago
|
||
Is there any other way to get at the public key material for a cert? Displaying info about the public key is something embeders and/or mozilla may want to do one day.
|
Assignee | |
Comment 2•22 years ago
|
||
If we don't currently have implemented a way to obtain that information, then we don't need to do that before a freeze. We can do that at a later time using the following strategy: - add a new separate interface - make the implementation of nsIX509Cert QIable to the new interface
|
|
Assignee | |
Comment 3•22 years ago
|
||
Javi, can you please review?
|
|
||
Comment 4•22 years ago
|
||
Comment on attachment 99679 [details] [diff] [review] Patch v1 r=javi
Attachment #99679 -
Flags: review+
|
|
Assignee | |
Comment 5•22 years ago
|
||
Alec, can you please review?
|
||
Comment 6•22 years ago
|
||
Comment on attachment 99679 [details] [diff] [review] Patch v1 sr=rpotts@netscape.com
Attachment #99679 -
Flags: superreview+
|
|
Assignee | |
Comment 7•22 years ago
|
||
Fixed on trunk.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
|
||
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•