Make sure all suffixes in PSL are lower-case, and update test to check this

RESOLVED FIXED in mozilla30

Status

()

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: gerv, Assigned: weppos)

Tracking

unspecified
mozilla30
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

We should make all the suffixes in the PSL lower-case. And we should update the PSL test (or add a new test) to make sure this is the case, so we don't regress. 

Gerv
weppos: are you able to take this on?

Gerv
(Assignee)

Comment 2

5 years ago
+1. I also vote to update the site to mention this information.
weppos: so can you prepare a patch? :-)

Gerv

Comment 4

5 years ago
Created attachment 8371530 [details]
txt file of all contracted ICANN new TLD strings from 7/14/13-2/5/2014, with the strings converted to lower case.

The ICANN New TLDs are all coming in lower now but previous were mixed case.

Might be worth doing the whole new TLD section over which is the primary culprit of mixed case.

I've attached the entire new TLD contracted list through 2/5, lc'd as a text file attachment.
(Assignee)

Comment 5

5 years ago
Created attachment 8371844 [details] [diff] [review]
CASE.patch

Here's the patch for TOKYO. Apparently, this was the only leftover.

Still need to add the tests, but I need some time to investigate how to create and run them locally. In the meanwhile, I suggest to checkin this commit.
Attachment #8371844 - Flags: review?(gerv)
Comment on attachment 8371844 [details] [diff] [review]
CASE.patch

r=gerv.

Gerv
Attachment #8371844 - Flags: review?(gerv) → review+
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/e85f83f352de
Assignee: nobody → weppos
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/e85f83f352de
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in before you can comment on or make changes to this bug.