[fix] Fix ftp port

unshiftio · Oct 30, 2015 · 4c52b7b · 4c52b7b
1 parent 0fad9a3
commit 4c52b7b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/index.js b/index.js
@@ -25,7 +25,7 @@ module.exports = function required(port, protocol) {
     return port !== 443;
 
     case 'ftp':
-    return port !== 22;
+    return port !== 21;
 
     case 'gopher':
     return port !== 70;

diff --git a/test.js b/test.js
@@ -72,11 +72,11 @@ describe('requires-port', function () {
     assume(required(8080, 'wss://www.google.com')).true();
   });
 
-  it('does not require port 22 for ftp', function () {
-    assume(required('22', 'ftp')).false();
-    assume(required(22, 'ftp')).false();
-    assume(required(22, 'ftp://')).false();
-    assume(required(22, 'ftp://www.google.com')).false();
+  it('does not require port 21 for ftp', function () {
+    assume(required('21', 'ftp')).false();
+    assume(required(21, 'ftp')).false();
+    assume(required(21, 'ftp://')).false();
+    assume(required(21, 'ftp://www.google.com')).false();
 
     assume(required('8080', 'ftp')).true();
     assume(required(8080, 'ftp')).true();