After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 136186 - Not translating URL for forecast to lowercase
Not translating URL for forecast to lowercase
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: gweather
2.5.x
Other Linux
: High major
: ---
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-03-04 15:33 UTC by Bill Merriam
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: 2.6.next
GNOME version: ---

Attachments
Fixes a memory leak and makes the forecast work. (1.60 KB, patch)
2004-03-04 23:12 UTC, Chris Lahey 		none Details | Review

Description Bill Merriam 2004-03-04 15:33:23 UTC 
Using 2.5.6-1 rpm from fedora core 2 test 1 the applet is sending to
http://weather.noaa.gov/pub/data/forecasts/zone/ with the state and zone
name in upper case.  It gets back a 404 Not Found.  The web server is case
sensitive.  The url must be all lower case.
Comment 1 Luis Villa 2004-03-04 19:56:50 UTC 
That's pretty nasty. Thanks for tracking down the specific cause of
the bug, Bill.
Comment 2 Chris Lahey 2004-03-04 23:12:39 UTC 
Created attachment 25181 [details] [review]
Fixes a memory leak and makes the forecast work.
Comment 3 Dennis Smit 2004-03-05 09:57:02 UTC 
Thanks a lot Chris for patching this and Bill thanks for
reporting great work!

______
2004-03-05  Dennis Smit  <ds@nerds-incorporated.org>
                                                                     
                                                                     
    Submitting patch by:                                             
                                                                     
                             Chris Lahey 
<clahey@rigger.boston.ximian.com>                                    
                                                                     
                                     
* weather.c (iwin_start_open): This was not using the return value of
g_ascii_strdown which allocates a new value.                         
                                                                     
                                                 Fixes bug #136186.