Add .es to IDN TLD whitelist

VERIFIED FIXED

Status

()

Core
Networking
--
enhancement
VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: gerv, Assigned: gerv)

Tracking

({intl, verified1.8.1.12})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

883 bytes, patch
Samuel Sidler (old account; do not CC)
: approval1.8.1.12+
beltzner
: approval1.9+
Details | Diff | Splinter Review
(Assignee)

Description

10 years ago
NIC: https://www.nic.es/
Regulations: https://www.nic.es/descargas/normativa.pdf
Character list: https://www.nic.es/descargas/FAQs.pdf

Seems fine to me. They only allow middle dot in the combo l<dot>l, like the Catalans. They don't bundle accented and non-accented domains, but we don't require that.

Gerv
(Assignee)

Comment 1

10 years ago
Checking in tld-idn-policy-list.html;
/www/mozilla-org/html/projects/security/tld-idn-policy-list.html,v  <--  tld-idn-policy-list.html
new revision: 1.16; previous revision: 1.15
done

Gerv
(Assignee)

Comment 2

10 years ago
Created attachment 289919 [details] [diff] [review]
Patch v.1

This patch applies to the branch, and to the trunk (with an offset). Requesting approval for both. 

This patch adds the ".es" TLD to the IDN whitelist.

Gerv
Assignee: nobody → gerv
Status: NEW → ASSIGNED
Attachment #289919 - Flags: approval1.9?
Attachment #289919 - Flags: approval1.8.1.11?
Comment on attachment 289919 [details] [diff] [review]
Patch v.1

a=beltzner for drivers
Attachment #289919 - Flags: approval1.9? → approval1.9+
(Assignee)

Comment 4

10 years ago
Checking in modules/libpref/src/init/all.js;
/cvsroot/mozilla/modules/libpref/src/init/all.js,v  <--  all.js
new revision: 3.707; previous revision: 3.706
done

Leaving bug open for branch approval.

Gerv
Fixed on trunk, we use keywords for branch
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Comment on attachment 289919 [details] [diff] [review]
Patch v.1

Approved for 1.8.1.12; a=ss for release-drivers. Please land as soon as possible.
Attachment #289919 - Flags: approval1.8.1.12? → approval1.8.1.12+
(Assignee)

Comment 7

10 years ago
Checking in modules/libpref/src/init/all.js;
/cvsroot/mozilla/modules/libpref/src/init/all.js,v  <--  all.js
new revision: 3.585.2.57; previous revision: 3.585.2.56
done

Gerv
Keywords: fixed1.8.1
(Assignee)

Updated

10 years ago
Keywords: fixed1.8.1 → fixed1.8.1.12
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.12pre) Gecko/20080128
BonEcho/2.0.0.12pre:

mozilla@mozilla-qa:~/Desktop/firefox/greprefs$ cat all.js | grep "whitelist.es"
pref("network.IDN.whitelist.es", true);

http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=&subdir=mozilla/modules/libpref/src/init&command=DIFF_FRAMESET&root=&file=all.js&rev1=3.585.2.56&rev2=3.585.2.57

Verified FIXED on the 1.8.1.12 branch; replacing fixed1.8.1.12 keyword with verified1.8.1.12
Keywords: fixed1.8.1.12 → intl, verified1.8.1.12
Verified on trunk with today's builds (Windows, Mac, Linux), by inspecting all.js
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.