Compare commits

..

2 Commits

Author SHA1 Message Date
Rafal Makagon
da59670d8e Increment version 2019-06-14 16:35:20 +02:00
Rafal Makagon
ed1049b543 Write down unexpected responses from http client 2019-06-14 16:32:07 +02:00
2 changed files with 8 additions and 3 deletions

View File

@@ -2,7 +2,7 @@ from setuptools import setup, find_packages
setup( setup(
name="galaxy.plugin.api", name="galaxy.plugin.api",
version="0.33.1", version="0.34",
description="GOG Galaxy Integrations Python API", description="GOG Galaxy Integrations Python API",
author='Galaxy team', author='Galaxy team',
author_email='galaxy@gog.com', author_email='galaxy@gog.com',

View File

@@ -4,6 +4,7 @@ from http import HTTPStatus
import aiohttp import aiohttp
import certifi import certifi
import logging
from galaxy.api.errors import ( from galaxy.api.errors import (
AccessDenied, AuthenticationRequired, BackendTimeout, BackendNotAvailable, BackendError, NetworkError, AccessDenied, AuthenticationRequired, BackendTimeout, BackendNotAvailable, BackendError, NetworkError,
@@ -21,9 +22,9 @@ class HttpClient:
async def close(self): async def close(self):
await self._session.close() await self._session.close()
async def request(self, method, *args, **kwargs): async def request(self, method, url, *args, **kwargs):
try: try:
response = await self._session.request(method, *args, **kwargs) response = await self._session.request(method, url, *args, **kwargs)
except asyncio.TimeoutError: except asyncio.TimeoutError:
raise BackendTimeout() raise BackendTimeout()
except aiohttp.ServerDisconnectedError: except aiohttp.ServerDisconnectedError:
@@ -33,6 +34,8 @@ class HttpClient:
except aiohttp.ContentTypeError: except aiohttp.ContentTypeError:
raise UnknownBackendResponse() raise UnknownBackendResponse()
except aiohttp.ClientError: except aiohttp.ClientError:
logging.exception(
"Caught exception while running {} request for {}".format(method, url))
raise UnknownError() raise UnknownError()
if response.status == HTTPStatus.UNAUTHORIZED: if response.status == HTTPStatus.UNAUTHORIZED:
raise AuthenticationRequired() raise AuthenticationRequired()
@@ -45,6 +48,8 @@ class HttpClient:
if response.status >= 500: if response.status >= 500:
raise BackendError() raise BackendError()
if response.status >= 400: if response.status >= 400:
logging.warning(
"Got status {} while running {} request for {}".format(response.status, method, url))
raise UnknownError() raise UnknownError()
return response return response