Merge pull request #237 from chatziko/master

Allow sending an ack with multiple data items (making it consistent with emit)
rikulo · Nov 19, 2021 · e6d0cc1 · e6d0cc1
2 parents 97e3467 + 286e831
commit e6d0cc1
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/lib/src/socket.dart b/lib/src/socket.dart
@@ -280,16 +280,25 @@ class Socket extends EventEmitter {
   /// @api private
   Function ack(id) {
     var sent = false;
-    return (_) {
+    return (dynamic data) {
       // prevent double callbacks
       if (sent) return;
       sent = true;
-      _logger.fine('sending ack $_');
+      _logger.fine('sending ack $data');
+
+      var sendData = <dynamic>[];
+      if (data is ByteBuffer || data is List<int>) {
+        sendData.add(data);
+      } else if (data is Iterable) {
+        sendData.addAll(data);
+      } else if (data != null) {
+        sendData.add(data);
+      }
 
       packet({
         'type': ACK,
         'id': id,
-        'data': [_]
+        'data': sendData
       });
     };
   }