b4bac0f7a0
* Remove unnecessary exception re-wraps * Preserve exception chains on re-raise We slap "from cause" to almost all possible cases here. In some cases it could conceivably be better to do "from None" if we really want to hide the cause. However those should be in the minority, and "from cause" should be an improvement over the corresponding raise without a "from" in all cases anyway. The only case where we raise from None here is in plex, where the exception for an original invalid SSL cert is not the root cause for failure to validate a newly fetched one. Follow local convention on exception variable names if there is a consistent one, otherwise `err` to match with majority of codebase. * Fix mistaken re-wrap in homematicip_cloud/hap.py Missed the difference between HmipConnectionError and HmipcConnectionError. * Do not hide original error on plex new cert validation error Original is not the cause for the new one, but showing old in the traceback is useful nevertheless.
85 lines
2.9 KiB
Python
85 lines
2.9 KiB
Python
"""Config flow for National Weather Service (NWS) integration."""
|
|
import logging
|
|
|
|
import aiohttp
|
|
from pynws import SimpleNWS
|
|
import voluptuous as vol
|
|
|
|
from homeassistant import config_entries, core, exceptions
|
|
from homeassistant.const import CONF_API_KEY, CONF_LATITUDE, CONF_LONGITUDE
|
|
from homeassistant.helpers import config_validation as cv
|
|
from homeassistant.helpers.aiohttp_client import async_get_clientsession
|
|
|
|
from . import base_unique_id
|
|
from .const import CONF_STATION, DOMAIN # pylint:disable=unused-import
|
|
|
|
_LOGGER = logging.getLogger(__name__)
|
|
|
|
|
|
async def validate_input(hass: core.HomeAssistant, data):
|
|
"""Validate the user input allows us to connect.
|
|
|
|
Data has the keys from DATA_SCHEMA with values provided by the user.
|
|
"""
|
|
latitude = data[CONF_LATITUDE]
|
|
longitude = data[CONF_LONGITUDE]
|
|
api_key = data[CONF_API_KEY]
|
|
station = data.get(CONF_STATION)
|
|
|
|
client_session = async_get_clientsession(hass)
|
|
ha_api_key = f"{api_key} homeassistant"
|
|
nws = SimpleNWS(latitude, longitude, ha_api_key, client_session)
|
|
|
|
try:
|
|
await nws.set_station(station)
|
|
except aiohttp.ClientError as err:
|
|
_LOGGER.error("Could not connect: %s", err)
|
|
raise CannotConnect from err
|
|
|
|
return {"title": nws.station}
|
|
|
|
|
|
class ConfigFlow(config_entries.ConfigFlow, domain=DOMAIN):
|
|
"""Handle a config flow for National Weather Service (NWS)."""
|
|
|
|
VERSION = 1
|
|
CONNECTION_CLASS = config_entries.CONN_CLASS_CLOUD_POLL
|
|
|
|
async def async_step_user(self, user_input=None):
|
|
"""Handle the initial step."""
|
|
errors = {}
|
|
if user_input is not None:
|
|
await self.async_set_unique_id(
|
|
base_unique_id(user_input[CONF_LATITUDE], user_input[CONF_LONGITUDE])
|
|
)
|
|
self._abort_if_unique_id_configured()
|
|
try:
|
|
info = await validate_input(self.hass, user_input)
|
|
user_input[CONF_STATION] = info["title"]
|
|
return self.async_create_entry(title=info["title"], data=user_input)
|
|
except CannotConnect:
|
|
errors["base"] = "cannot_connect"
|
|
except Exception: # pylint: disable=broad-except
|
|
_LOGGER.exception("Unexpected exception")
|
|
errors["base"] = "unknown"
|
|
|
|
data_schema = vol.Schema(
|
|
{
|
|
vol.Required(CONF_API_KEY): str,
|
|
vol.Required(
|
|
CONF_LATITUDE, default=self.hass.config.latitude
|
|
): cv.latitude,
|
|
vol.Required(
|
|
CONF_LONGITUDE, default=self.hass.config.longitude
|
|
): cv.longitude,
|
|
vol.Optional(CONF_STATION): str,
|
|
}
|
|
)
|
|
|
|
return self.async_show_form(
|
|
step_id="user", data_schema=data_schema, errors=errors
|
|
)
|
|
|
|
|
|
class CannotConnect(exceptions.HomeAssistantError):
|
|
"""Error to indicate we cannot connect."""
|