Version 2.4:

- Fix: WebSocketClient hat TLS-Pakete falsch zerlegt.
Nall-chan · Jun 14, 2020 · c31cc55 · c31cc55
1 parent c112639
commit c31cc55
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 12 deletions.
diff --git a/README.md b/README.md
@@ -1,5 +1,5 @@
 [![Version](https://img.shields.io/badge/Symcon-PHPModul-red.svg)](https://www.symcon.de/service/dokumentation/entwicklerbereich/sdk-tools/sdk-php/)
-[![Version](https://img.shields.io/badge/Modul%20Version-2.3-blue.svg)]()
+[![Version](https://img.shields.io/badge/Modul%20Version-2.4-blue.svg)]()
 [![License](https://img.shields.io/badge/License-CC%20BY--NC--SA%204.0-green.svg)](https://creativecommons.org/licenses/by-nc-sa/4.0/)  
 [![Version](https://img.shields.io/badge/Symcon%20Version-4.3%20%3E-green.svg)](https://www.symcon.de/forum/threads/30857-IP-Symcon-4-3-%28Stable%29-Changelog)
 [![StyleCI](https://styleci.io/repos/104255893/shield?style=flat)](https://styleci.io/repos/104255893)  
@@ -72,6 +72,9 @@ Details sind in der Dokumentation der jeweiligen Module beschrieben.
 
 ### 2. Changlog
 
+Version 2.4:  
+ - Fix: WebSocketClient hat TLS-Pakete falsch zerlegt.  
+
 Version 2.3:  
  - Fix: DHCP-Sniffer hat während des Symcon Systemstart auf falsche DHCP Requests reagiert.
 

diff --git a/WebSocketClient/module.php b/WebSocketClient/module.php
@@ -13,9 +13,9 @@
  * @package       Network
  * @file          module.php
  * @author        Michael Tröger <[email protected]>
- * @copyright     2017 Michael Tröger
+ * @copyright     2020 Michael Tröger
  * @license       https://creativecommons.org/licenses/by-nc-sa/4.0/ CC BY-NC-SA 4.0
- * @version       1.0
+ * @version       2.4
  */
 
 /**
@@ -222,7 +222,7 @@ public function ApplyChanges()
         }
         $OldState = $this->State;
         $this->SendDebug(__FUNCTION__, 'OldState:' . $OldState, 0);
-        if ((($OldState != WebSocketState::unknow) and ($OldState != WebSocketState::Connected)) or ($OldState == WebSocketState::init)) {
+        if ((($OldState != WebSocketState::unknow) and ( $OldState != WebSocketState::Connected)) or ( $OldState == WebSocketState::init)) {
             return;
         }
 
@@ -745,7 +745,7 @@ public function ReceiveData($JSONString)
                         break;
                     }
                 }
-                if (strlen($TLSData) > 0) {
+                if (strlen($TLSData) == 0) {
                     $this->TLSReceiveBuffer = '';
                 } else {
                     //$this->SendDebug('Receive TLS Part', $TLSData, 0);

diff --git a/WebSocketServer/README.md b/WebSocketServer/README.md
@@ -87,9 +87,9 @@ Implementierung eines Server mit Websocket Protokoll in IPS.
 ## 6. PHP-Befehlsreferenz
 
 ```php
-bool WSS_SendPing(integer $InstanzeID, string $ClientIP, string $Text);
+bool WSS_SendPing(integer $InstanzeID, string $ClientIP, int $ClientPort, string $Text);
 ```
- Senden die in `$Text` übergeben Daten als Payload eines Ping an die `$ClientIP` eines WebSocket-Clients.
+ Senden die in `$Text` übergeben Daten als Payload eines Ping an den in `$ClientIP` & `$ClientPort` angegeben WebSocket-Client.  
  Der Rückgabewert ist `True`, wenn der Client verbunden ist und den Ping beantwortet.  
 
 

diff --git a/WebSocketServer/module.php b/WebSocketServer/module.php
@@ -1109,12 +1109,12 @@ public function KeepAlive()
      * Versendet einen Ping an einen Client.
      *
      * @param string $ClientIP   Die IP-Adresse des Client.
-     * @param string $ClientPort Der Port des Client.
+     * @param int $ClientPort Der Port des Client.
      * @param string $Text       Der Payload des Ping.
      *
      * @return bool True bei Erfolg, im Fehlerfall wird eine Warnung und false ausgegeben.
      */
-    public function SendPing(string $ClientIP, string $ClientPort, string $Text)
+    public function SendPing(string $ClientIP, int $ClientPort, string $Text)
     {
         $Client = $this->Multi_Clients->GetByIpPort(new Websocket_Client($ClientIP, $ClientPort));
         if ($Client === false) {

diff --git a/library.json b/library.json
@@ -7,7 +7,7 @@
         "version": "4.3",
         "date": 1498241677
     },
-    "version": "2.3",
-    "build": 300,
-    "date": 1590515480 
+    "version": "2.4",
+    "build": 320,
+    "date": 1592137828 
 }