Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 3 years ago

Closed 3 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@…> 3 years ago.
Patch to nws-alert.pl
results.orig.log (102 bytes) - added by Brett T. Warden <4ddab56a@…> 3 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@…> 3 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@…> 3 years ago.
Output of "mythbackend --version"

Download all attachments as: .zip

Change History (5)

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

Patch to nws-alert.pl

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

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

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

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

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

Output of "mythbackend --version"

comment:1 Changed 3 years ago by Github

  • Milestone changed from unknown to 0.25
  • Resolution set to fixed
  • Status changed from new to closed

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

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.