If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Add a variant for importPKCS12File

RESOLVED WONTFIX

Status

()

Core
Security: PSM
RESOLVED WONTFIX
4 years ago
2 months ago

People

(Reporter: allstars, Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
ARM
All
Points:
---

Firefox Tracking Flags

(tracking-b2g:backlog)

Details

(Whiteboard: [FT:RIL][psm-backlog])

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Currently nsIX509CertDB.importPKCS12File will invoke some XUL dialog to get password for the PKCS12 file, but on B2G we try to avoid use XUL here.

So we need to create another variant for importPKCS12File, either
1. Add another parameter for password.
2. Dispatch some browser element (for dialog) event to Gaia and let Gaia handle the UI part for inputing password.

Please also see the previous discussing thread on dev-b2g. [1]


[1]: https://groups.google.com/forum/#!msg/mozilla.dev.b2g/YKUBFge_qxc/LQJYv9CU3YIJ

Updated

4 years ago
Whiteboard: RIL5/29

Updated

4 years ago
Whiteboard: RIL5/29 → RN5/29
Component: Libraries → Security: PSM
Product: NSS → Core
Version: trunk → Trunk

Updated

4 years ago
Whiteboard: RN5/29 → RN6/7
(Reporter)

Comment 1

4 years ago
Hi, Brian
Does the new method should be in nsIX509CertDB2.idl?

From nsIX509CertDB2.idl I saw the comment
/**
 * This represents a service to access and manipulate 
 * X.509 certificates stored in a database through methods
 * not in nsIX509CertDB, which is *frozen*
 *
 */

By frozen does it we cannot modify nsIX509CertDB.idl?

Thanks
The comment is obsolete. All interfaces are unfrozen since Firefox 4.
(Reporter)

Comment 3

4 years ago
Created attachment 759672 [details] [diff] [review]
Add password for importPKCS12File
(Reporter)

Comment 4

4 years ago
Try https://tbpl.mozilla.org/?tree=Try&rev=4958128c43b1
(Reporter)

Comment 5

4 years ago
Comment on attachment 759672 [details] [diff] [review]
Add password for importPKCS12File

Hi, Brian
Can you review this patch with additinal password parameter for importPKCS12File for me ?

Thanks
Attachment #759672 - Flags: review?(bsmith)
(Reporter)

Comment 6

4 years ago
Hi, bsmith, review ping?
Or if the patch is wrong in somethere, feel free to tell me and cancel r?. :)
(Reporter)

Comment 7

4 years ago
Comment on attachment 759672 [details] [diff] [review]
Add password for importPKCS12File

Canceling r? as I think for Web it should use prompt to get the password if needed.

I should try if we could create another component like NSSDialog on B2G.
Attachment #759672 - Flags: review?(brian)

Updated

4 years ago
Whiteboard: RN6/7

Updated

4 years ago
blocking-b2g: --- → 1.3+
(Reporter)

Updated

4 years ago
Target Milestone: --- → 1.3 Sprint 6 - 12/6

Updated

4 years ago
Whiteboard: [FT:RIL]
(Reporter)

Comment 8

4 years ago
Remove 1.3+ and Target Mile Stone for we will do this in Wifi, Bug 917102.
blocking-b2g: 1.3+ → ---
Target Milestone: 1.3 Sprint 6 - 12/6 → ---

Updated

3 years ago
blocking-b2g: --- → backlog
(Reporter)

Updated

3 years ago
Assignee: allstars.chh → nobody
(Assignee)

Updated

3 years ago
blocking-b2g: backlog → ---
tracking-b2g: --- → backlog
This would be a great way to move towards PSM code not causing any dialogs to open.
Whiteboard: [FT:RIL] → [FT:RIL][psm-backlog]
(Reporter)

Updated

2 months ago
Status: NEW → RESOLVED
Last Resolved: 2 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.