mirror of https://github.com/home-assistant/core
clean up clicksend (#17723)
This commit is contained in:
parent
398ea40189
commit
37a667c2de
|
@ -7,51 +7,41 @@ https://home-assistant.io/components/notify.clicksend/
|
|||
import json
|
||||
import logging
|
||||
|
||||
from aiohttp.hdrs import CONTENT_TYPE
|
||||
import requests
|
||||
import voluptuous as vol
|
||||
from aiohttp.hdrs import CONTENT_TYPE
|
||||
|
||||
import homeassistant.helpers.config_validation as cv
|
||||
from homeassistant.components.notify import (
|
||||
PLATFORM_SCHEMA, BaseNotificationService)
|
||||
from homeassistant.const import (
|
||||
CONF_API_KEY, CONF_RECIPIENT, CONF_SENDER, CONF_USERNAME,
|
||||
CONTENT_TYPE_JSON)
|
||||
import homeassistant.helpers.config_validation as cv
|
||||
|
||||
_LOGGER = logging.getLogger(__name__)
|
||||
|
||||
BASE_API_URL = 'https://rest.clicksend.com/v3'
|
||||
|
||||
DEFAULT_SENDER = 'hass'
|
||||
TIMEOUT = 5
|
||||
|
||||
HEADERS = {CONTENT_TYPE: CONTENT_TYPE_JSON}
|
||||
|
||||
|
||||
def validate_sender(config):
|
||||
"""Set the optional sender name if sender name is not provided."""
|
||||
if CONF_SENDER in config:
|
||||
return config
|
||||
config[CONF_SENDER] = DEFAULT_SENDER
|
||||
return config
|
||||
|
||||
|
||||
PLATFORM_SCHEMA = vol.Schema(
|
||||
vol.All(PLATFORM_SCHEMA.extend({
|
||||
vol.Required(CONF_USERNAME): cv.string,
|
||||
vol.Required(CONF_API_KEY): cv.string,
|
||||
vol.Required(CONF_RECIPIENT, default=[]):
|
||||
vol.All(cv.ensure_list, [cv.string]),
|
||||
vol.Optional(CONF_SENDER): cv.string,
|
||||
}), validate_sender))
|
||||
vol.Optional(CONF_SENDER, default=DEFAULT_SENDER): cv.string,
|
||||
}),))
|
||||
|
||||
|
||||
def get_service(hass, config, discovery_info=None):
|
||||
"""Get the ClickSend notification service."""
|
||||
print("#### ", config)
|
||||
if _authenticate(config) is False:
|
||||
_LOGGER.exception("You are not authorized to access ClickSend")
|
||||
if not _authenticate(config):
|
||||
_LOGGER.error("You are not authorized to access ClickSend")
|
||||
return None
|
||||
|
||||
return ClicksendNotificationService(config)
|
||||
|
||||
|
||||
|
@ -60,10 +50,10 @@ class ClicksendNotificationService(BaseNotificationService):
|
|||
|
||||
def __init__(self, config):
|
||||
"""Initialize the service."""
|
||||
self.username = config.get(CONF_USERNAME)
|
||||
self.api_key = config.get(CONF_API_KEY)
|
||||
self.recipients = config.get(CONF_RECIPIENT)
|
||||
self.sender = config.get(CONF_SENDER)
|
||||
self.username = config[CONF_USERNAME]
|
||||
self.api_key = config[CONF_API_KEY]
|
||||
self.recipients = config[CONF_RECIPIENT]
|
||||
self.sender = config[CONF_SENDER]
|
||||
|
||||
def send_message(self, message="", **kwargs):
|
||||
"""Send a message to a user."""
|
||||
|
@ -77,16 +67,17 @@ class ClicksendNotificationService(BaseNotificationService):
|
|||
})
|
||||
|
||||
api_url = "{}/sms/send".format(BASE_API_URL)
|
||||
|
||||
resp = requests.post(
|
||||
api_url, data=json.dumps(data), headers=HEADERS,
|
||||
auth=(self.username, self.api_key), timeout=5)
|
||||
resp = requests.post(api_url,
|
||||
data=json.dumps(data),
|
||||
headers=HEADERS,
|
||||
auth=(self.username, self.api_key),
|
||||
timeout=TIMEOUT)
|
||||
if resp.status_code == 200:
|
||||
return
|
||||
|
||||
obj = json.loads(resp.text)
|
||||
response_msg = obj['response_msg']
|
||||
response_code = obj['response_code']
|
||||
|
||||
if resp.status_code != 200:
|
||||
response_msg = obj.get('response_msg')
|
||||
response_code = obj.get('response_code')
|
||||
_LOGGER.error("Error %s : %s (Code %s)", resp.status_code,
|
||||
response_msg, response_code)
|
||||
|
||||
|
@ -94,11 +85,11 @@ class ClicksendNotificationService(BaseNotificationService):
|
|||
def _authenticate(config):
|
||||
"""Authenticate with ClickSend."""
|
||||
api_url = '{}/account'.format(BASE_API_URL)
|
||||
resp = requests.get(
|
||||
api_url, headers=HEADERS, auth=(config.get(CONF_USERNAME),
|
||||
config.get(CONF_API_KEY)), timeout=5)
|
||||
|
||||
resp = requests.get(api_url,
|
||||
headers=HEADERS,
|
||||
auth=(config[CONF_USERNAME],
|
||||
config[CONF_API_KEY]),
|
||||
timeout=TIMEOUT)
|
||||
if resp.status_code != 200:
|
||||
return False
|
||||
|
||||
return True
|
||||
|
|
Loading…
Reference in New Issue