search

burnnat / ha-hdhomerun

HDHomeRun integration for Home Assistant.
MIT License
15 stars 6 forks source link

Gracefully handle empty model names #4

Closed burnnat closed 4 years ago

burnnat commented 4 years ago

Apparently some devices don't report a model name. Sample error:

Error while setting up platform hdhomerun
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 158, in _async_setup_platform
    await asyncio.wait_for(asyncio.shield(task), SLOW_SETUP_MAX_WAIT)
  File "/usr/local/lib/python3.7/asyncio/tasks.py", line 442, in wait_for
    return fut.result()
  File "/config/custom_components/hdhomerun/sensor.py", line 46, in async_setup_entry
    'model': adapter.get_model_str(),
  File "/usr/local/lib/python3.7/site-packages/hdhr/adapter.py", line 178, in get_model_str
    return ascii_str(result)
  File "/usr/local/lib/python3.7/site-packages/hdhr/utility.py", line 7, in ascii_str
    return str(value, encoding='ascii')
TypeError: decoding to str: need a bytes-like object, NoneType found