Opened 13 years ago

Closed 13 years ago

#9799 closed Patch - Bug Fix (fixed)

MythWeather NWS Alerts script fails to retrieve alerts

Reported by: Brett T. Warden <4ddab56a@…> Owned by: beirdo
Priority: minor Milestone: 0.25
Component: Plugin - MythWeather Version: 0.24-fixes
Severity: medium Keywords:
Cc: Ticket locked: no

Description

The nws-alerts.pl script included with MythWeather? needs to be updated to match the new URL and information format provided by the National Weather Service. I've made appropriate changes and submitted a pull request:

https://github.com/MythTV/mythtv/pull/4

URL has changed from: http://www.weather.gov/alerts/$state.cap to: http://www.weather.gov/alerts/$state.php?x=0 (actually redirects to http://alerts.weather.gov/cap/$state.php?x=0 )

Attachments (4)

nws-alert.pl.patch (2.9 KB) - added by Brett T. Warden <4ddab56a@…> 13 years ago.
Patch to nws-alert.pl
results.orig.log (102 bytes) - added by Brett T. Warden <4ddab56a@…> 13 years ago.
Results of running original script with parameters "-u ENG -d /tmp 22071"
results.new.log (401 bytes) - added by Brett T. Warden <4ddab56a@…> 13 years ago.
Results of running new script with parameters "-u ENG -d /tmp 22071"
myth_version.txt (698 bytes) - added by Brett T. Warden <4ddab56a@…> 13 years ago.
Output of "mythbackend --version"

Download all attachments as: .zip

Change History (5)

Changed 13 years ago by Brett T. Warden <4ddab56a@…>

Attachment: nws-alert.pl.patch added

Patch to nws-alert.pl

Changed 13 years ago by Brett T. Warden <4ddab56a@…>

Attachment: results.orig.log added

Results of running original script with parameters "-u ENG -d /tmp 22071"

Changed 13 years ago by Brett T. Warden <4ddab56a@…>

Attachment: results.new.log added

Results of running new script with parameters "-u ENG -d /tmp 22071"

Changed 13 years ago by Brett T. Warden <4ddab56a@…>

Attachment: myth_version.txt added

Output of "mythbackend --version"

comment:1 Changed 13 years ago by Github

Milestone: unknown0.25
Resolution: fixed
Status: newclosed

Merge pull request #4 from bwarden/patch-1

Updated to parse new format of CAP messages from alerts.weather.gov

Closes #9799

Branch: master Changeset: fb6d707d74ffd21aa1f0dc2c7bccbf94c9acb3c8

Note: See TracTickets for help on using tickets.