From d487ef52521646df62f442d214d62f9ef4696cd0 Mon Sep 17 00:00:00 2001
From: Tom van der Lee <tom@vanderlee.io>
Date: Fri, 18 Nov 2022 13:42:59 +0100
Subject: Handle client connections gracefully

---
 .pre-commit-config.yaml |  6 +++---
 ttun/client.py          | 51 ++++++++++++++++++++++++++++++-------------------
 2 files changed, 34 insertions(+), 23 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index c5c9a1e..5cecfd3 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -6,19 +6,19 @@ repos:
       - id: end-of-file-fixer
       - id: trailing-whitespace
   - repo: https://github.com/psf/black
-    rev: 21.12b0
+    rev: 22.10.0
     hooks:
       - id: black
         types_or:
           - python
   - repo: https://github.com/asottile/reorder_python_imports
-    rev: v2.7.1
+    rev: v3.9.0
     hooks:
       - id: reorder-python-imports
         types_or:
           - python
   - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: 'v2.5.1'
+    rev: v2.7.1
     hooks:
       - id: prettier
         types_or:
diff --git a/ttun/client.py b/ttun/client.py
index 671ac59..6d05da7 100644
--- a/ttun/client.py
+++ b/ttun/client.py
@@ -11,6 +11,8 @@ from typing import Optional
 from uuid import uuid4
 
 import websockets
+from aiohttp import ClientConnectionError
+from aiohttp import ClientError
 from aiohttp import ClientSession
 from aiohttp import DummyCookieJar
 from websockets import WebSocketClientProtocol
@@ -66,14 +68,14 @@ class Client:
         while True:
             try:
                 request: RequestData = await self.receive()
-                await self.proxyRequest(
+                await self.proxy_request(
                     request=request, on_response=lambda response: self.send(response)
                 )
 
             except ConnectionClosed:
                 break
 
-    async def proxyRequest(
+    async def proxy_request(
         self,
         request: RequestData,
         on_response: Callable[[ResponseData], Awaitable] = None,
@@ -92,24 +94,33 @@ class Client:
             )
 
             start = perf_counter()
-            response = await session.request(
-                method=request["method"],
-                url=f'http://localhost:{self.port}{request["path"]}',
-                headers=request["headers"],
-                data=b64decode(request["body"].encode()),
-                allow_redirects=False,
-            )
-            end = perf_counter()
-
-            response_data = ResponseData(
-                status=response.status,
-                headers=[
-                    (key, value)
-                    for key, value in response.headers.items()
-                    if key.lower() not in ["transfer-encoding", "content-encoding"]
-                ],
-                body=b64encode(await response.read()).decode(),
-            )
+            try:
+                response = await session.request(
+                    method=request["method"],
+                    url=f'http://localhost:{self.port}{request["path"]}',
+                    headers=request["headers"],
+                    data=b64decode(request["body"].encode()),
+                    allow_redirects=False,
+                )
+                end = perf_counter()
+
+                response_data = ResponseData(
+                    status=response.status,
+                    headers=[
+                        (key, value)
+                        for key, value in response.headers.items()
+                        if key.lower() not in ["transfer-encoding", "content-encoding"]
+                    ],
+                    body=b64encode(await response.read()).decode(),
+                )
+            except ClientError as e:
+                end = perf_counter()
+
+                response_data = ResponseData(
+                    status=(504 if isinstance(e, ClientConnectionError) else 502),
+                    headers=[("content-type", "text/plain")],
+                    body=b64encode(str(e).encode()).decode(),
+                )
 
             if on_response is not None:
                 await on_response(response_data)
-- 
cgit v1.2.3