Merge pull request #132 from manuelm/master

Don't append payload per byte unless necessary
Textalk · Feb 24, 2021 · caa1610 · caa1610
2 parents b932494 + 3389629
commit caa1610
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/lib/Base.php b/lib/Base.php
@@ -207,12 +207,15 @@ protected function sendFragment(bool $final, string $payload, string $opcode, bo
                 $mask .= chr(rand(0, 255));
             }
             $data .= $mask;
-        }
 
-        // Append payload to frame:
-        for ($i = 0; $i < $payload_length; $i++) {
-            $data .= ($masked === true) ? $payload[$i] ^ $mask[$i % 4] : $payload[$i];
+            // Append payload to frame:
+            for ($i = 0; $i < $payload_length; $i++) {
+                $data .= $payload[$i] ^ $mask[$i % 4];
+            }
+        } else {
+            $data .= $payload;
         }
+
         $this->write($data);
         $this->logger->debug("Sent '{$opcode}' frame", [
             'opcode' => $opcode,