v5 protocol: rename ERROR packet to CONNECT_ERROR

socketio/asyncio_client.py

@@ -450,7 +450,7 @@ class AsyncClient(client.Client):
             elif pkt.packet_type == packet.BINARY_EVENT or \
                     pkt.packet_type == packet.BINARY_ACK:
                 self._binary_packet = pkt
-            elif pkt.packet_type == packet.ERROR:
+            elif pkt.packet_type == packet.CONNECT_ERROR:
                 await self._handle_error(pkt.namespace, pkt.data)
             else:
                 raise ValueError('Unknown packet type.')

socketio/asyncio_server.py

@@ -427,7 +427,8 @@ class AsyncServer(server.Server):
                     packet.DISCONNECT, data=fail_reason, namespace=namespace))
             elif namespace != '/':
                 await self._send_packet(sid, packet.Packet(
-                    packet.ERROR, data=fail_reason, namespace=namespace))
+                    packet.CONNECT_ERROR, data=fail_reason,
+                    namespace=namespace))
             self.manager.disconnect(sid, namespace)
             if namespace == '/' and sid in self.environ:  # pragma: no cover
                 del self.environ[sid]
@@ -541,8 +542,8 @@ class AsyncServer(server.Server):
             elif pkt.packet_type == packet.BINARY_EVENT or \
                     pkt.packet_type == packet.BINARY_ACK:
                 self._binary_packet[sid] = pkt
-            elif pkt.packet_type == packet.ERROR:
+            elif pkt.packet_type == packet.CONNECT_ERROR:
-                raise ValueError('Unexpected ERROR packet.')
+                raise ValueError('Unexpected CONNECT_ERROR packet.')
             else:
                 raise ValueError('Unknown packet type.')
 

socketio/client.py

@@ -605,7 +605,7 @@ class Client(object):
             elif pkt.packet_type == packet.BINARY_EVENT or \
                     pkt.packet_type == packet.BINARY_ACK:
                 self._binary_packet = pkt
-            elif pkt.packet_type == packet.ERROR:
+            elif pkt.packet_type == packet.CONNECT_ERROR:
                 self._handle_error(pkt.namespace, pkt.data)
             else:
                 raise ValueError('Unknown packet type.')

socketio/packet.py

@@ -3,9 +3,9 @@ import json as _json
 
 import six
 
-(CONNECT, DISCONNECT, EVENT, ACK, ERROR, BINARY_EVENT, BINARY_ACK) = \
+(CONNECT, DISCONNECT, EVENT, ACK, CONNECT_ERROR, BINARY_EVENT, BINARY_ACK) = \
     (0, 1, 2, 3, 4, 5, 6)
-packet_names = ['CONNECT', 'DISCONNECT', 'EVENT', 'ACK', 'ERROR',
+packet_names = ['CONNECT', 'DISCONNECT', 'EVENT', 'ACK', 'CONNECT_ERROR',
                 'BINARY_EVENT', 'BINARY_ACK']
 
 

socketio/server.py

@@ -625,7 +625,8 @@ class Server(object):
                     packet.DISCONNECT, data=fail_reason, namespace=namespace))
             elif namespace != '/':
                 self._send_packet(sid, packet.Packet(
-                    packet.ERROR, data=fail_reason, namespace=namespace))
+                    packet.CONNECT_ERROR, data=fail_reason,
+                    namespace=namespace))
             self.manager.disconnect(sid, namespace)
             if namespace == '/' and sid in self.environ:  # pragma: no cover
                 del self.environ[sid]
@@ -729,8 +730,8 @@ class Server(object):
             elif pkt.packet_type == packet.BINARY_EVENT or \
                     pkt.packet_type == packet.BINARY_ACK:
                 self._binary_packet[sid] = pkt
-            elif pkt.packet_type == packet.ERROR:
+            elif pkt.packet_type == packet.CONNECT_ERROR:
-                raise ValueError('Unexpected ERROR packet.')
+                raise ValueError('Unexpected CONNECT_ERROR packet.')
             else:
                 raise ValueError('Unknown packet type.')
 

tests/common/test_packet.py

@@ -87,7 +87,7 @@ class TestPacket(unittest.TestCase):
 
     def test_invalid_binary_packet(self):
         with pytest.raises(ValueError):
-            packet.Packet(packet_type=packet.ERROR, data=b'123')
+            packet.Packet(packet_type=packet.CONNECT_ERROR, data=b'123')
 
     def test_encode_namespace(self):
         pkt = packet.Packet(