Fix DenonAvr discovery of Denon DN-500AV

[starkillerOG]

Proposed change

Fix DenonAvr discovery of Denon DN-500AV

Type of change

• Dependency upgrade
• Bugfix (non-breaking change which fixes an issue)
• New integration (thank you!)
• New feature (which adds functionality to an existing integration)
• Breaking change (fix/feature causing existing functionality to break)
• Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

Config Flow

Additional information

• This PR fixes or closes issue: fixes Denon AVR is not being detected by the config flow #37484
• This PR is related to issue:
• Link to documentation pull request:

Checklist

• The code change is tested and works locally.
• Local tests pass. Your PR cannot be merged unless tests pass
• There is no commented out code in this PR.
• I have followed the development checklist
• The code has been formatted using Black (black --fast homeassistant tests)
• Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

• The manifest file has all fields filled out correctly.
  Updated and included derived files by running: python3 -m script.hassfest.
• New or updated dependencies have been added to requirements_all.txt.
  Updated by running python3 -m script.gen_requirements_all.
• Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

• No score or internal
• 🥈 Silver
• 🥇 Gold
• 🏆 Platinum

[probot-home-assistant]

Hey there @scarface-4711, mind taking a look at this pull request as its been labeled with an integration (denonavr) you are listed as a codeowner for? Thanks!
_{^{(message by CodeOwnersMention)}}

[starkillerOG]

To fully fix this issue, this PR in the parent lib needs to be merged: ol-iver/denonavr#145
The required PR is now merged

[frenck]

Converted the PR to draft until upstream is ready.

[ol-iver]

@starkillerOG there is version 0.9.4 which includes your fix now

[starkillerOG]

@bdraco how can I remove the waiting-for-upstream label, or can that only be done by maintainers?

[starkillerOG]

Test failure is unrelated, the problem is in this file:
tests/components/unifi/test_device_tracker.py

[bdraco]

Tested on my production install. All ok

[netforceatg]

Running Home Assistant 0.113.2

Have used the denonavr component to control my Zone 2 of my Denon AVR-X5200W for a few years. Been really happy with it

However with the updates, I have removed the denonavr from configuration.yaml and added it through the web intergration. However HA does not find it on the IP i put in(local browser on my computer finds the denon webpage control fine).

Any advice? (I know the avr have network connectivity)

[starkillerOG]

Have you read the error message?
Disconnects mains power of the receiver wait 10 seconds, plug it back in

[netforceatg]

Thanks, I've leaved the reciever unplugged and retested. Same result.

Here is the errorlog:
2020-07-31 19:36:23 ERROR (ThreadPoolExecutor-6_0) [DenonSSDP] Error occurred during evaluation of SCPD XML: not well-formed (invalid token): line 24, column 79
2020-07-31 19:36:23 ERROR (ThreadPoolExecutor-7_1) [DenonAVR] Host 192.168.2.15 returned malformed XML for end point /goform/formMainZone_MainZoneXml.xml: not well-formed (invalid token): line 3, column 84
2020-07-31 19:36:23 ERROR (ThreadPoolExecutor-6_1) [DenonAVR] Host 192.168.2.15 returned malformed XML for end point /goform/formMainZone_MainZoneXml.xml: not well-formed (invalid token): line 3, column 84
2020-07-31 19:36:23 WARNING (ThreadPoolExecutor-6_1) [DenonAVR] Receiver name could not be determined. Using standard name: Denon AVR.
2020-07-31 19:36:23 ERROR (ThreadPoolExecutor-8_2) [DenonAVR] Host 192.168.2.15 returned malformed XML for end point /goform/formMainZone_MainZoneXml.xml: not well-formed (invalid token): line 3, column 84
2020-07-31 19:36:23 ERROR (MainThread) [homeassistant.components.denonavr.receiver] Missing receiver information: manufacturer 'None', name 'Denon AVR', model 'None', type 'avr-x'
2020-07-31 19:36:32 ERROR (SyncWorker_4) [homeassistant.components.scrape.sensor] Unable to extract data from HTML
2020-07-31 19:36:45 ERROR (MainThread) [frontend.js.latest.202007160] http://192.168.2.30:83/frontend_latest/chunk.cf565e41b8a8dce0642a.js:133:238 Uncaught TypeError: Cannot read property 'header' of null
2020-07-31 19:36:46 ERROR (MainThread) [frontend.js.latest.202007160] http://192.168.2.30:83/frontend_latest/chunk.cf565e41b8a8dce0642a.js:133:238 Uncaught TypeError: Cannot read property 'header' of null
2020-07-31 19:36:47 ERROR (MainThread) [frontend.js.latest.202007160] http://192.168.2.30:83/frontend_latest/chunk.cf565e41b8a8dce0642a.js:133:238 Uncaught TypeError: Cannot read property 'header' of null
2020-07-31 19:37:02 ERROR (SyncWorker_4) [homeassistant.components.scrape.sensor] Unable to extract data from HTML

[MartinHjelmare]

Please open an issue if you suspect a bug. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels

Merged PRs should not be used for support or bug reports. Thanks!