fix(commands): 502 error on unsupported command (#185)

* fix(commands): 502 error on unsupported command Fixes: #184 * test: update test assertion
QuorumDMS · Jan 6, 2020 · b2b1b2a · b2b1b2a
1 parent 81fa7fc
commit b2b1b2a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/commands/index.js b/src/commands/index.js
@@ -45,7 +45,7 @@ class FtpCommands {
     log.trace({command: logCommand}, 'Handle command');
 
     if (!REGISTRY.hasOwnProperty(command.directive)) {
-      return this.connection.reply(402, 'Command not allowed');
+      return this.connection.reply(502, 'Command not allowed');
     }
 
     if (_.includes(this.blacklist, command.directive)) {

diff --git a/test/commands/index.spec.js b/test/commands/index.spec.js
@@ -90,7 +90,7 @@ describe('FtpCommands', function () {
       return commands.handle('bad')
       .then(() => {
         expect(mockConnection.reply.callCount).to.equal(1);
-        expect(mockConnection.reply.args[0][0]).to.equal(402);
+        expect(mockConnection.reply.args[0][0]).to.equal(502);
       });
     });