Last Comment Bug 435874 - Ensure byte length limits of l10n messages, notably in pipnss.properties
: Ensure byte length limits of l10n messages, notably in pipnss.properties
Status: NEW
:
Product: Mozilla Localizations
Classification: Client Software
Component: Infrastructure (show other bugs)
: unspecified
: All All
: -- normal (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
:
:
Mentors:
http://bonsai.mozilla.org/cvsblame.cg...
: 685264 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-05-27 07:32 PDT by Axel Hecht [:Pike]
Modified: 2012-07-11 03:31 PDT (History)
4 users (show)
l10n: in‑testsuite?
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
ad-hoc test (906 bytes, text/plain)
2008-06-13 07:16 PDT, Axel Hecht [:Pike]
no flags Details

Description Axel Hecht [:Pike] 2008-05-27 07:32:46 PDT
Looking at bug 317630 and bug 435789, we should probably have a better grip on those length limits. Before moving this into localization or testing, are only the commented entries in pipnss.properties affected or are there more?

It'd be good to have both a source and a runtime check.
Comment 1 Axel Hecht [:Pike] 2008-05-27 09:59:43 PDT
One way to reproduce a cut off name is to 

1. set Firefox to have a master password 
2. Go to an url that requires login
3. the popup dialog appears with a shortened string

according to Damjan on #l10n.
Comment 2 Axel Hecht [:Pike] 2008-06-13 06:33:41 PDT
Kai, can you help out here with a comment?
Comment 3 Kai Engert (:kaie) 2008-06-13 06:46:03 PDT
The commented strings in pipnss.properties are all I'm aware of.

I'm not sure what "source" check you have in mind. Something in the tools that localizers use?

Regarding runtime check, we could check the loaded strings just before calling PK11_ConfigurePKCS11, but what do you propose to do?
Comment 4 Axel Hecht [:Pike] 2008-06-13 07:16:14 PDT
Created attachment 324962 [details]
ad-hoc test

This is an ad-hoc test, and sadly, we're hitting this all over.

I'll file individual bugs per locale.

af:
        Fips140TokenDescription:        2 bytes
be:
        Fips140TokenDescription:        1 bytes
ca:
        InternalToken:  5 bytes
        PrivateTokenDescription:        5 bytes
        LibraryDescription:     3 bytes
        Fips140TokenDescription:        12 bytes
cs:
        LibraryDescription:     1 bytes
es-AR:
        TokenDescription:       3 bytes
        InternalToken:  5 bytes
        PrivateTokenDescription:        5 bytes
        LibraryDescription:     8 bytes
        Fips140TokenDescription:        12 bytes
eu:
        LibraryDescription:     1 bytes
        Fips140TokenDescription:        1 bytes
fr:
        LibraryDescription:     5 bytes
        Fips140TokenDescription:        8 bytes
fy-NL:
        Fips140TokenDescription:        4 bytes
ga-IE:
        TokenDescription:       1 bytes
        LibraryDescription:     6 bytes
hu:
        Fips140SlotDescription: 2 bytes
        Fips140TokenDescription:        4 bytes
hy-AM:
        LibraryDescription:     7 bytes
id:
        InternalToken:  2 bytes
        PrivateTokenDescription:        2 bytes
        Fips140TokenDescription:        9 bytes
it:
        LibraryDescription:     1 bytes
        Fips140TokenDescription:        8 bytes
kn:
        TokenDescription:       1 bytes
ku:
        Fips140SlotDescription: 2 bytes
        LibraryDescription:     2 bytes
        Fips140TokenDescription:        7 bytes
lt:
        Fips140TokenDescription:        2 bytes
mn:
        LibraryDescription:     6 bytes
        Fips140TokenDescription:        7 bytes
nb-NO:
        Fips140TokenDescription:        2 bytes
nl:
        Fips140TokenDescription:        4 bytes
nn-NO:
        Fips140TokenDescription:        16 bytes
pt-PT:
        InternalToken:  4 bytes
        PrivateTokenDescription:        4 bytes
        Fips140TokenDescription:        11 bytes
ro:
        LibraryDescription:     2 bytes
        Fips140TokenDescription:        8 bytes
rw:
        InternalToken:  6 bytes
        PrivateTokenDescription:        6 bytes
        LibraryDescription:     1 bytes
sk:
        Fips140TokenDescription:        2 bytes
sq:
        InternalToken:  1 bytes
        PrivateTokenDescription:        1 bytes
        Fips140TokenDescription:        8 bytes
sr:
        Fips140SlotDescription: 2 bytes
        Fips140TokenDescription:        10 bytes
uk:
        InternalToken:  9 bytes
        Fips140TokenDescription:        16 bytes
zu:
        TokenDescription:       2 bytes
        InternalToken:  4 bytes
        PrivateTokenDescription:        4 bytes
        LibraryDescription:     15 bytes
Comment 5 Axel Hecht [:Pike] 2008-06-13 07:17:39 PDT
Over to l10n.
Comment 6 Axel Hecht [:Pike] 2011-09-07 12:54:17 PDT
*** Bug 685264 has been marked as a duplicate of this bug. ***
Comment 7 Axel Hecht [:Pike] 2012-07-11 03:31:45 PDT
arky, you asked about a test, here's a python file that does that.

Note You need to log in before you can comment on or make changes to this bug.