home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
71.05k stars 29.71k forks source link

Meteoalarm doesn't get information from feed #122112

Open tschoiss opened 1 month ago

tschoiss commented 1 month ago

The problem

I have 2 alarams set. One for "Steyr-Land" and this works. The second area is "Steyr (Stadt)". I copied this directly from the feed, but it is never fetched correctly from the integration. There are never alarms shown in Home Assistant. My guess is, that if there is a blank in the area and/or a bracket, something will went wrong.

What version of Home Assistant Core has the issue?

core-2024.7.2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

MeteoAlarm

Link to integration documentation on our website

https://www.home-assistant.io/integrations/meteoalarm/

Diagnostics information

No response

Example YAML snippet

binary_sensor:
  - platform: meteoalarm
    country: "austria"
    name: "Meteoalarm SR"
    province: "Steyr (Stadt)"
    language: de-DE

Anything in the logs that might be useful for us?

No response

Additional information

Example entry in the feed for "Steyr (Stadt)": `

EMMA_ID AT402 Steyr (Stadt) Heatwarning 2024-07-17T07:02:39+00:00 2024-07-18T21:59:59+00:00 2024-07-17T07:02:39+00:00 2024-07-17T22:00:00+00:00 Likely Moderate Future Public Alert Actual 2.49.0.0.40.0.AT.-20240717090239_d1_402 meteoalarm.org https://meteoalarm.org 2024-07-17T07:02:39Z https://feeds.meteoalarm.org/api/v1/warnings/feeds-austria/5fbe62a5-e7e3-4e41-a0ed-78f04fa2ff30?index_info=1&index_area=0&index_geocode=0 Yellow High-temperature Warning issued for Austria - Steyr (Stadt) 2024-07-17T07:02:39Z `
home-assistant[bot] commented 1 month ago

Hey there @rolfberkenbosch, mind taking a look at this issue as it has been labeled with an integration (meteoalarm) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `meteoalarm` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign meteoalarm` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


meteoalarm documentation meteoalarm source (message by IssueLinks)

marcelvriend commented 1 month ago

It's a bug upstream not escaping the regex input. You can work around it by escaping the braces yourself in the YAML, like: province: "Steyr \\(Stadt\\)".

Waits for fix: https://github.com/rolfberkenbosch/meteoalert-api/pull/33