Skip to content

Commit

Permalink
Merge pull request #361 from fronzbot/add-user-agent-everywhere
Browse files Browse the repository at this point in the history
Add user agent to all headers
  • Loading branch information
fronzbot authored Aug 2, 2020
2 parents 5ebf278 + cb43f17 commit fd104c0
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
9 changes: 7 additions & 2 deletions blinkpy/auth.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,12 @@
from urllib3.util.retry import Retry
from blinkpy import api
from blinkpy.helpers import util
from blinkpy.helpers.constants import BLINK_URL, LOGIN_ENDPOINT, TIMEOUT
from blinkpy.helpers.constants import (
BLINK_URL,
DEFAULT_USER_AGENT,
LOGIN_ENDPOINT,
TIMEOUT,
)

_LOGGER = logging.getLogger(__name__)

Expand Down Expand Up @@ -53,7 +58,7 @@ def header(self):
"""Return authorization header."""
if self.token is None:
return None
return {"TOKEN_AUTH": self.token}
return {"TOKEN_AUTH": self.token, "user-agent": DEFAULT_USER_AGENT}

def create_session(self, opts=None):
"""Create a session for blink communication."""
Expand Down
2 changes: 1 addition & 1 deletion tests/test_auth.py
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ def test_response_bad_json(self):
def test_header(self):
"""Test header data."""
self.auth.token = "bar"
expected_header = {"TOKEN_AUTH": "bar"}
expected_header = {"TOKEN_AUTH": "bar", "user-agent": const.DEFAULT_USER_AGENT}
self.assertDictEqual(self.auth.header, expected_header)

def test_header_no_token(self):
Expand Down

0 comments on commit fd104c0

Please sign in to comment.