ED/EDMarketConnector
1
0
Fork 0
mirror of https://github.com/EDCD/EDMarketConnector.git synced 2025-04-17 17:42:20 +03:00
Code Issues Projects Releases Wiki Activity

companion.Session: Don't double-initaliase requests_session

Browse Source 
I forgot to remove the `requests.Session()` call in `start_frontier_auth()`.
And upon removing that we need a `None` trap to make mypy happy.
This commit is contained in:
Athanasius 2022-12-16 15:40:48 +00:00
parent 729016cf38
commit 5070c3686f
No known key found for this signature in database
GPG Key ID: 772697E181BB2767
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
companion.py
View File

@ -652,9 +652,10 @@ class Session(object):
def start_frontier_auth(self, access_token: str) -> None: def start_frontier_auth(self, access_token: str) -> None:
"""Start an oAuth2 session.""" """Start an oAuth2 session."""
logger.debug('Starting session') logger.debug('Starting session')
self.requests_session = requests.Session() if self.requests_session is not None:
self.requests_session.headers['Authorization'] = f'Bearer {access_token}' self.requests_session.headers['Authorization'] = f'Bearer {access_token}'
self.requests_session.headers['User-Agent'] = user_agent self.requests_session.headers['User-Agent'] = user_agent
self.state = Session.STATE_OK self.state = Session.STATE_OK
def login(self, cmdr: Optional[str] = None, is_beta: Optional[bool] = None) -> bool: def login(self, cmdr: Optional[str] = None, is_beta: Optional[bool] = None) -> bool: