Add new line delimiter to all IPCProvider writes

Author: fselmo

tests/core/providers/test_ipc_provider.py

@@ -9,6 +9,7 @@
 )
 import time
 from unittest.mock import (
+    Mock,
     patch,
 )
 
@@ -185,3 +186,14 @@ def test_web3_auto_gethdev(request_mocker):
 
     assert "extraData" not in block
     assert block.proofOfAuthorityData == b"\xff" * 33
+
+
+def test_ipc_provider_write_messages_end_with_new_line_delimiter(jsonrpc_ipc_pipe_path):
+    provider = IPCProvider(pathlib.Path(jsonrpc_ipc_pipe_path), timeout=3)
+    provider._socket.sock = Mock()
+    provider._socket.sock.recv.return_value = b'{"id":1, "result": {}}\n'
+
+    provider.make_request("method", [])
+
+    request_data = b'{"jsonrpc": "2.0", "method": "method", "params": [], "id": 0}'
+    provider._socket.sock.sendall.assert_called_with(request_data + b"\n")

web3/providers/ipc.py

@@ -164,7 +164,7 @@ def __str__(self) -> str:
     def _make_request(self, request: bytes) -> RPCResponse:
         with self._lock, self._socket as sock:
             try:
-                sock.sendall(request)
+                sock.sendall(request + b"\n")
             except BrokenPipeError:
                 # one extra attempt, then give up
                 sock = self._socket.reset()