Broadlink Switch Not Working on 0.109.6

[levipereira]

The problem

After upgrade HAS from 0.108.9 to 0.109.6 the error below is raised. Invalid config for [switch.broadlink]: not a valid value for dictionary value @ data['host'].

Environment

• Home Assistant Core release with the issue: 0.109.6
• Last working Home Assistant Core release (if known): 0.108.9
• Operating environment (Home Assistant/Supervised/Docker/venv): venv
• Integration causing this issue: Broadlink
• Link to integration documentation on our website: https://www.home-assistant.io/integrations/broadlink/

  Problem-relevant configuration.yaml

 - platform: broadlink
   host: rm_mini
   mac: redacted
   timeout: 15
   type: rm_mini
   friendly_name: Broadlink

Traceback/Error logs

Failed config
  switch.broadlink:
    - Invalid config for [switch.broadlink]: not a valid value for dictionary value @ data['host']. Got 'rm_mini'. (See ?, line ?).
    - platform: broadlink
      friendly_name: Broadlink
      host: rm_mini
      mac: redacted
      timeout: 15
      type: rm_mini

Additional information

Is working on homeassistant==0.108.9 without issues.

[probot-home-assistant[bot]]

broadlink documentation broadlink source _{^{(message by IssueLinks)}}

[probot-home-assistant[bot]]

Hey there @danielhiversen, @felipediel, mind taking a look at this issue as its been labeled with a integration (broadlink) you are listed as a codeowner for? Thanks! _{^{(message by CodeOwnersMention)}}

[felipediel]

rm_mini is not a valid hostname. Please use the IP address instead.

[levipereira]

I tried rm_mini.mydomain.net and also does not worked. rm mini it become a reserved word on switch configuration?

[felipediel]

The problem is the underscore, which is not in line with the standard for hostnames, but Broadlink engineers decided to adopt it. I will modify the validation method to accept it. It will take a while to merge. You can use the IP address for now.