Merge pull request #10 from mattsch/bugfix/login

Make the error message a little clearer when login is needed

Author: mattsch

custom_components/badnest/__init__.py

@@ -3,8 +3,7 @@
 from homeassistant.helpers import config_validation as cv
 
 from .api import NestAPI
-from .const import DOMAIN, CONF_ISSUE_TOKEN, CONF_COOKIE, CONF_USER_ID, \
-        CONF_ACCESS_TOKEN
+from .const import DOMAIN, CONF_ISSUE_TOKEN, CONF_COOKIE
 
 CONFIG_SCHEMA = vol.Schema(
     {

custom_components/badnest/api.py

@@ -111,6 +111,7 @@ def __init__(self,
         self.thermostats = set()
         self.temperature_sensors = set()
         self.protects = set()
+
         self.login()
         self._get_devices()
         self.update()
@@ -151,7 +152,13 @@ def login(self):
         except RequestException as e:
             _LOGGER.error(e)
 
-        access_token = r.json()['access_token']
+        try:
+            access_token = r.json()['access_token']
+        except KeyError:
+            _LOGGER.error(f"{r.json()['error']}: {r.json()['detail']}")
+            _LOGGER.error("Invalid cookie or issue_token. Please see:")
+            _LOGGER.error("https://github.com/mattsch/badnest#configuration")
+            raise
 
         headers = {
             'User-Agent': USER_AGENT,

custom_components/badnest/const.py

@@ -1,5 +1,3 @@
 DOMAIN = 'badnest'
 CONF_ISSUE_TOKEN = 'issue_token'
 CONF_COOKIE = 'cookie'
-CONF_USER_ID = 'user_id'
-CONF_ACCESS_TOKEN = 'access_token'

info.md

@@ -35,7 +35,7 @@ will never be as reliable as the original API
 ```yaml
 badnest:
   issue_token: "https://accounts.google.com/o/oauth2/iframerpc....."
-  cookie: "OCAK=......"
+  cookie: "......"
 
 climate:
   - platform: badnest