fetch: Block ports 4190 & 6679 (#3059)

nodejs · Apr 6, 2024 · 043d8f1 · 043d8f1
1 parent 9130c85
commit 043d8f1
Show file tree

Hide file tree

Showing 3 changed files with 89 additions and 85 deletions.
diff --git a/lib/web/fetch/constants.js b/lib/web/fetch/constants.js
@@ -14,8 +14,8 @@ const badPorts = [
   '87', '95', '101', '102', '103', '104', '109', '110', '111', '113', '115', '117', '119', '123', '135', '137',
   '139', '143', '161', '179', '389', '427', '465', '512', '513', '514', '515', '526', '530', '531', '532',
   '540', '548', '554', '556', '563', '587', '601', '636', '989', '990', '993', '995', '1719', '1720', '1723',
-  '2049', '3659', '4045', '5060', '5061', '6000', '6566', '6665', '6666', '6667', '6668', '6669', '6697',
-  '10080'
+  '2049', '3659', '4045', '4190', '5060', '5061', '6000', '6566', '6665', '6666', '6667', '6668', '6669', '6679',
+  '6697', '10080'
 ]
 
 const badPortsSet = new Set(badPorts)

diff --git a/test/wpt/tests/fetch/api/request/request-bad-port.any.js b/test/wpt/tests/fetch/api/request/request-bad-port.any.js
@@ -3,90 +3,92 @@
 // list of bad ports according to
 // https://fetch.spec.whatwg.org/#port-blocking
 var BLOCKED_PORTS_LIST = [
-    1,    // tcpmux
-    7,    // echo
-    9,    // discard
-    11,   // systat
-    13,   // daytime
-    15,   // netstat
-    17,   // qotd
-    19,   // chargen
-    20,   // ftp-data
-    21,   // ftp
-    22,   // ssh
-    23,   // telnet
-    25,   // smtp
-    37,   // time
-    42,   // name
-    43,   // nicname
-    53,   // domain
-    69,   // tftp
-    77,   // priv-rjs
-    79,   // finger
-    87,   // ttylink
-    95,   // supdup
-    101,  // hostriame
-    102,  // iso-tsap
-    103,  // gppitnp
-    104,  // acr-nema
-    109,  // pop2
-    110,  // pop3
-    111,  // sunrpc
-    113,  // auth
-    115,  // sftp
-    117,  // uucp-path
-    119,  // nntp
-    123,  // ntp
-    135,  // loc-srv / epmap
-    137,  // netbios-ns
-    139,  // netbios-ssn
-    143,  // imap2
-    161,  // snmp
-    179,  // bgp
-    389,  // ldap
-    427,  // afp (alternate)
-    465,  // smtp (alternate)
-    512,  // print / exec
-    513,  // login
-    514,  // shell
-    515,  // printer
-    526,  // tempo
-    530,  // courier
-    531,  // chat
-    532,  // netnews
-    540,  // uucp
-    548,  // afp
-    554,  // rtsp
-    556,  // remotefs
-    563,  // nntp+ssl
-    587,  // smtp (outgoing)
-    601,  // syslog-conn
-    636,  // ldap+ssl
-    989,  // ftps-data
-    990,  // ftps
-    993,  // ldap+ssl
-    995,  // pop3+ssl
-    1719, // h323gatestat
-    1720, // h323hostcall
-    1723, // pptp
-    2049, // nfs
-    3659, // apple-sasl
-    4045, // lockd
-    5060, // sip
-    5061, // sips
-    6000, // x11
-    6566, // sane-port
-    6665, // irc (alternate)
-    6666, // irc (alternate)
-    6667, // irc (default)
-    6668, // irc (alternate)
-    6669, // irc (alternate)
-    6697, // irc+tls
-    10080, // amanda
+  1,    // tcpmux
+  7,    // echo
+  9,    // discard
+  11,   // systat
+  13,   // daytime
+  15,   // netstat
+  17,   // qotd
+  19,   // chargen
+  20,   // ftp-data
+  21,   // ftp
+  22,   // ssh
+  23,   // telnet
+  25,   // smtp
+  37,   // time
+  42,   // name
+  43,   // nicname
+  53,   // domain
+  69,   // tftp
+  77,   // priv-rjs
+  79,   // finger
+  87,   // ttylink
+  95,   // supdup
+  101,  // hostriame
+  102,  // iso-tsap
+  103,  // gppitnp
+  104,  // acr-nema
+  109,  // pop2
+  110,  // pop3
+  111,  // sunrpc
+  113,  // auth
+  115,  // sftp
+  117,  // uucp-path
+  119,  // nntp
+  123,  // ntp
+  135,  // loc-srv / epmap
+  137,  // netbios-ns
+  139,  // netbios-ssn
+  143,  // imap2
+  161,  // snmp
+  179,  // bgp
+  389,  // ldap
+  427,  // afp (alternate)
+  465,  // smtp (alternate)
+  512,  // print / exec
+  513,  // login
+  514,  // shell
+  515,  // printer
+  526,  // tempo
+  530,  // courier
+  531,  // chat
+  532,  // netnews
+  540,  // uucp
+  548,  // afp
+  554,  // rtsp
+  556,  // remotefs
+  563,  // nntp+ssl
+  587,  // smtp (outgoing)
+  601,  // syslog-conn
+  636,  // ldap+ssl
+  989,  // ftps-data
+  990,  // ftps
+  993,  // ldap+ssl
+  995,  // pop3+ssl
+  1719, // h323gatestat
+  1720, // h323hostcall
+  1723, // pptp
+  2049, // nfs
+  3659, // apple-sasl
+  4045, // lockd
+  4190, // sieve
+  5060, // sip
+  5061, // sips
+  6000, // x11
+  6566, // sane-port
+  6665, // irc (alternate)
+  6666, // irc (alternate)
+  6667, // irc (default)
+  6668, // irc (alternate)
+  6669, // irc (alternate)
+  6679, // osaut
+  6697, // irc+tls
+  10080, // amanda
 ];
 
 BLOCKED_PORTS_LIST.map(function(a){
-    promise_test(function(t){
-        return promise_rejects_js(t, TypeError, fetch("http://example.com:" + a))
-    }, 'Request on bad port ' + a + ' should throw TypeError.');
+  promise_test(function(t){
+      return promise_rejects_js(t, TypeError, fetch("http://example.com:" + a))
+  }, 'Request on bad port ' + a + ' should throw TypeError.');
 });
diff --git a/test/wpt/tests/websockets/Create-blocked-port.any.js b/test/wpt/tests/websockets/Create-blocked-port.any.js
@@ -79,13 +79,15 @@ async_test(t => {
   2049, // nfs
   3659, // apple-sasl
   4045, // lockd
+  4190, // sieve
   6000, // x11
   6566, // sane-port
   6665, // irc (alternate)
   6666, // irc (alternate)
   6667, // irc (default)
   6668, // irc (alternate)
   6669, // irc (alternate)
+  6679, // osaut
   6697, // irc+tls
   10080, // amanda
 ].forEach(blockedPort => {