84151 - CA name is blank (do you want to trust "") when accepting new CA's

Status: VERIFIED FIXED

(Core Graveyard :: Security: UI, defect, P2)

Description

[Henrik Gemal]

If you go to:
http://digitalid.verisign.com/test_ca_cps.html
and press Accept in the bottom of the page, the dialog getting shown contains 
the following text:
"Do you want to trust "" for the following purposes"

Comment 1

[John Unruh]

The CN (Common Name) is null, so that's probably why.

Comment 2

[Bob Lord]

I suppose we could put in a new string saying "There is no CN here, folks" when
it's null.

-> 2.1

Comment 3

[(Don't use, old account) - Kai Engert]

Attachment: Proposed patch, using message "Certificate Authority (unnamed)

Comment 4

[(Don't use, old account) - Kai Engert]

To apply the patch, go to mozilla root directoy and use
  patch -p0 < patchfile

Comment 5

[Stephane Saux]

kai, I think  that this string should come from the bundle.  Otherwise it
doesn't get localized. Let's have Sean comment on that.

Comment 6

[(Don't use, old account) - Kai Engert]

Good point, thanks for hinting. I think I already figured out how to do it, will
file another patch shortly.

Comment 7

[(Don't use, old account) - Kai Engert]

Attachment: Enhanced patch, now the message is stored in the bundle.

Comment 8

[(Don't use, old account) - Kai Engert]

David, please review.

Comment 9

[Stephane Saux]

r=ssaux
David review as well when you have a moment.

Comment 10

[Stephane Saux]

p2

Comment 11

[Stephane Saux]

-> kai

Comment 12

[David P. Drinan]

r=ddrinan.

Comment 13

[Christopher Blizzard (:blizzard)]

sr=blizzard

Comment 14

[John Unruh]

Mass assigning QA to ckritzer.

Comment 15

[Javier Delgadillo]

checked in patch for Kai.

Comment 16

[Stephane Saux]

keywords->verifyme

Comment 17

[John Unruh]

Verified on 8/2 WinNT trunk.