Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix an Ambient PWS exception when location info is missing #21220

Merged
merged 1 commit into from Feb 20, 2019

Conversation

Projects
None yet
4 participants
@bachya
Copy link
Contributor

bachya commented Feb 19, 2019

Description:

A user pointed out to me that if an Ambient customer doesn't specify a device name or location name, those fields are removed from the socket response. Currently, that'll throw an unhandled exception. This PR fixes it by defaulting to using the MAC address if no name is provided.

Related issue (if applicable): N/A

Pull request in home-assistant.io with documentation (if applicable): N/A

Example entry for configuration.yaml (if applicable):

ambient_station:
  api_key: !secret ambient_pws_api_key
  app_key: !secret ambient_pws_app_key

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.

@bachya bachya added this to the 0.88.0 milestone Feb 19, 2019

@bachya bachya self-assigned this Feb 19, 2019

@wafflebot wafflebot bot added the in progress label Feb 19, 2019

@bachya bachya force-pushed the bachya:ambient-location-fix branch from bd1c223 to e1fcd12 Feb 19, 2019

@bachya bachya force-pushed the bachya:ambient-location-fix branch from e1fcd12 to 37718c3 Feb 19, 2019

@MartinHjelmare MartinHjelmare merged commit 7e06d03 into home-assistant:dev Feb 20, 2019

4 checks passed

Hound No violations found. Woof!
cla-bot Everyone involved has signed the CLA
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 92.803%
Details

@wafflebot wafflebot bot removed the in progress label Feb 20, 2019

@bachya bachya deleted the bachya:ambient-location-fix branch Feb 20, 2019

@balloob balloob referenced this pull request Feb 20, 2019

Merged

0.88.0 #21238

thibmaek added a commit to thibmaek/home-assistant that referenced this pull request Feb 21, 2019

Merge remote-tracking branch 'upstream/dev' into feature/discogs-v2
* upstream/dev: (948 commits)
  Prevent partial custom component overlays (home-assistant#21070)
  Alarm trigger support for Point (home-assistant#21207)
  Upgrade opensensemap-api to 0.1.4 (home-assistant#21240)
  Remove constraint from regex (home-assistant#21239)
  Updated frontend to 20190220.0
  Support OpenWRT 18.06 in luci device tracker (home-assistant#21236)
  Bump zigpy (home-assistant#21203)
  Fix bug in ZHA and tweak non sensor channel logic (home-assistant#21234)
  Don't dispatch to components when there are no channels for ZHA sensors (home-assistant#21223)
  Add zone and reps for Xiaomi vacuum (home-assistant#19777)
  Fix an Ambient PWS exception when location info is missing (home-assistant#21220)
  Add self to integration sensor and utility_meter (home-assistant#21226)
  Prevent invalid context from crashing (home-assistant#21231)
  Update pyhomematic to 0.1.56 (home-assistant#21227)
  Fix the build (home-assistant#21229)
  Updated frontend to 20190219.0
  Add yeelight flow action support (home-assistant#21195)
  Refactor ZHA listeners into channels (home-assistant#21196)
  Fix Homematic IP Cloud configuration (home-assistant#21202)
  Push pyads to 3.0.7 (home-assistant#21216)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.