Closed
Bug 791769
Opened 13 years ago
Closed 11 years ago
Update PSL for .ve
Categories
(Core Graveyard :: Networking: Domain Lists, defect)
Core Graveyard
Networking: Domain Lists
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla33
People
(Reporter: gerv, Assigned: weppos)
Details
Attachments
(1 file, 2 obsolete files)
866 bytes,
patch
|
gerv
:
review+
|
Details | Diff | Splinter Review |
Paulo Ney de Souza writes:
Now the diff, with the problems of Paraguay and Venezuela.
Paulo Ney de Souza
diff -u list_old list_new
--- list_old 2012-07-04 20:00:39.000000000 -0300
+++ list_new 2012-07-14 23:56:59.697460844 -0300
@@ -5359,7 +5359,13 @@
belau.pw
// py : http://www.nic.py/faq_a.html#faq_b
-*.py
+py
+net.py
+com.py
+gov.py
+org.py
+edu.py
+coop.py
// qa : http://domains.qa/en/
qa
@@ -6255,7 +6261,13 @@
edu.vc
// ve : http://registro.nic.ve/nicve/registro/index.html
-*.ve
+ve
+com.ve
+net.ve
+org.ve
+info.ve
+co.ve
+web.ve
// vg : http://en.wikipedia.org/wiki/.vg
vg
Reporter | ||
Comment 1•13 years ago
|
||
I'm assuming that the links he gives will back up the changes. Simone: can you check?
Gerv
Assignee | ||
Comment 2•13 years ago
|
||
The diff for .PY confirms the patch I submitted in #779834.
However, my patch also includes .mil.py that is officially available.
About .VE, the link seems to include only these 2LD in the select, however I can see this page [1] also mentions gob.ve, mil.ve, edu.ve, tec.ve. I'm trying to find the complete list, assuming it exists somewhere.
[1] http://www.nic.ve/documentos
Reporter | ||
Comment 3•13 years ago
|
||
OK, let's switch this bug to just .ve.
Gerv
Assignee: nobody → weppos
Summary: Update PSL for .py and .ve → Update PSL for .ve
Assignee | ||
Comment 4•13 years ago
|
||
I contacted the .VE registrar, I'm waiting for a response.
Comment 5•13 years ago
|
||
(In reply to Simone Carletti from comment #4)
> I contacted the .VE registrar, I'm waiting for a response.
Hi,
The list of the 2LD are:
.com.ve
.co.ve
.org.ve
.info.ve
.web.ve
.net.ve
.edu.ve
.e12.ve
.mil.ve
.gov.ve
.ve like nic.ve (1LD)
You can see available for registry for through this url: registro.nic.ve
Regards
Assignee | ||
Comment 6•13 years ago
|
||
Created the patch according to the registry information.
Attachment #667865 -
Flags: review?(gerv)
Reporter | ||
Comment 7•13 years ago
|
||
Comment on attachment 667865 [details] [diff] [review]
Patch for .VE
r=gerv.
Gerv
Attachment #667865 -
Flags: review?(gerv) → review+
Reporter | ||
Comment 8•13 years ago
|
||
Comment 9•13 years ago
|
||
Unfortunately this caused xpcshell failures:
{
TEST-PASS | /Users/cltbld/talos-slave/test/build/xpcshell/tests/netwerk/test/unit/test_bug414122.js | [checkPublicSuffix : 62] gov.vc == gov.vc
TEST-PASS | /Users/cltbld/talos-slave/test/build/xpcshell/tests/netwerk/test/unit/test_bug414122.js | [checkPublicSuffix : 62] mil.vc == mil.vc
TEST-PASS | /Users/cltbld/talos-slave/test/build/xpcshell/tests/netwerk/test/unit/test_bug414122.js | [checkPublicSuffix : 62] edu.vc == edu.vc
TEST-UNEXPECTED-FAIL | xpcshell/head.js | [Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsIEffectiveTLDService.getPublicSuffixFromHost]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)" location: "JS frame :: /Users/cltbld/talos-slave/test/build/xpcshell/tests/netwerk/test/unit/test_bug414122.js :: checkPublicSuffix :: line 61" data: no]
WARNING: nsExceptionService ignoring thread destruction after shutdown: file ../../../xpcom/base/nsExceptionService.cpp, line 166
WARNING: OOPDeinit() without successful OOPInit(): file ../../../toolkit/crashreporter/nsExceptionHandler.cpp, line 2252
}
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=953079e94edd
Backout:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ca480aedae9d
Reporter | ||
Comment 10•13 years ago
|
||
Unfortunately, the patch was made with leading dots, which are invalid :-| Here is an updated patch.
Gerv
Attachment #667865 -
Attachment is obsolete: true
Reporter | ||
Comment 11•13 years ago
|
||
Assignee | ||
Comment 12•13 years ago
|
||
Sorry guys, my fault.
@ Gervase
On the PSL site http://publicsuffix.org/list/ I read
> Each rule lists a public suffix, with the subdomain portions separated by dots (.) as usual. A leading dot is optional.
Should we remove the
> A leading dot is optional.
portion?
Reporter | ||
Comment 13•13 years ago
|
||
I removed the "a leading dot is optional".
This wasn't closed but, searching the checkin log, it did get merged to central.
Gerv
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 14•11 years ago
|
||
This was merged in https://hg.mozilla.org/mozilla-central/rev/12b5f4b2f299 as noted in comment 11, there was a typo in the commit summary, which pointed to bug 761769 instead.
Reopening since gob.ve is missing (we have gov.ve instead). Comment 5 may have been incomplete, as comment 2 already listed gob.ve
The document [1] lists at «Disposiciones Específicas» (page 2) the tlds «gob.ve, mil.ve,edu.ve, tec.ve» (we miss gob.ve and tec.ve)
The «Disposiciones finales» point 3 states they are removing all registrations under gov.ve
However, there are still sites under it, such as http://www.tsj.gov.ve/
Sadly, I found no full tld list at https://registro.nic.ve/ and many old links are broken now.
In addition to those above, wikipedia also lists arts.ve, radio.ve and int.ve
Their whois output [3] confirms [4] the existance of arts.ve and int.ve (test with zamoralapelicula.arts.ve, iica.int.ve) but rejects radio.ve (it's a normal registration)
Whois output also suggests rec.ve, store.ve and firm.ve can be used for registration (cisco.rec.ve, cisco.store.ve, cisco.firm.ve).
In summary: Add gob.ve, tec.ve, arts.ve, int.ve, rec.ve, store.ve, firm.ve to the PSL
1- http://conatel.gob.ve/files/marco_legal/normativa_tecnica.pdf (also attached)
2- https://en.wikipedia.org/wiki/.ve
3- https://registro.nic.ve/modules/whois
4- There isn't any registrant information for the tld, but there is for the subdomain
5- https://en.wikipedia.org/w/index.php?title=.ve&diff=556960126&oldid=539273941
Google dork: site:.ve -site:.com.ve -site:.gob.ve -site:.org.ve -site:.web.ve -site:.info.ve -site:.net.ve -site:.edu.ve -site:.int.ve -site:e12.ve -site:.gov.ve -site:co.ve -site:mil.ve -site:arts.ve -site:usb.ve -site:ucv.ve -site:tec.ve -site:ula.ve -site:store.ve -site:rec.ve -site:firm.ve -site:luz.ve -site:ivic.ve -site:gobiernoenlinea.ve -site:reacciun.ve -site:cida.ve -site:internet.ve -site:nic.ve -site:ipv6.ve
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 15•11 years ago
|
||
Patch adding arts.ve, firm.ve, gob.ve, int.ve, rec.ve, store.ve and tec.ve per comment 14.
Attachment #668418 -
Attachment is obsolete: true
Attachment #8425683 -
Flags: review?(gerv)
Reporter | ||
Updated•11 years ago
|
Attachment #8425683 -
Flags: review?(gerv) → review+
Comment 16•11 years ago
|
||
Comment 17•11 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 13 years ago → 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•