Merge pull request mqttjs#5 from xadh00m/master

Do not shift the pubcomp fixed header by qos
satanatoly · Oct 15, 2015 · e0d3c73 · e0d3c73
2 parents f966ab2 + 7b12b09
commit e0d3c73
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/generate.js b/generate.js
@@ -288,8 +288,6 @@ function confirmation(opts) {
 
   if (type === 'pubrel')
     qos = 1
-  else if (type === 'pubcomp')
-    qos = 2
 
   // Check message ID
   if ('number' !== typeof id)

diff --git a/test.js b/test.js
@@ -522,12 +522,12 @@ testParseGenerate('pubrel', {
 testParseGenerate('pubcomp', {
     cmd: 'pubcomp'
   , retain: false
-  , qos: 2
+  , qos: 0
   , dup: false
   , length: 2
   , messageId: 2
 }, new Buffer([
-  116, 2, // Header
+  112, 2, // Header
   0, 2 // Message id
 ]))