Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.4.7

Bug #13845 Weather.com service requires link request in all xoap requests
Submitted: 2008-05-06 20:34 UTC Modified: 2008-05-10 15:43 UTC
From: mscannel Assigned: eru
Status: Closed Package: Services_Weather (version 1.4.2)
PHP Version: 5.2.0 OS: Linux 2.6.18
Roadmaps: (Not assigned)    
Subscription  


 [2008-05-06 20:34 UTC] mscannel (Mike Scannell)
Description: ------------ As of today (5/6/08), weather.com is requiring the xoap link request as part of the url. For example, this works: http://xoap.weather.com/weather/local/48164?link=xoap&prod=xoap&par=[removed]&key=[removed] This does not: http://xoap.weather.com/weather/local/48164?prod=xoap&par=[removed]&key=[removed] This can be fixed by including "&link=xoap" in all URLs. This is easily patched. Here is a diff: 248c248 < $url = "http://xoap.weather.com/weather/local/".$id."?prod=xoap&link=xoap&par=".$this->_partnerID."&key=".$this->_licenseKey; --- > $url = "http://xoap.weather.com/weather/local/".$id."?prod=xoap&par=".$this->_partnerID."&key=".$this->_licenseKey; Test script: --------------- weather.com-extensive.php should reproduce the problem Expected result: ---------------- Without the "link=xoap" in the URL, the returned data is always: <?xml version="1.0" encoding="ISO-8859-1"?> <error> <err type="8">Bad or missing query parameters in request.</err> </error>

Comments

 [2008-05-10 08:19 UTC] boigelot (Olivier Boigelot)
As mentionned by Mike the package could be easily fixed. I do it by modifying line 248 in Services/Weather/Weatherdotcom.php like by adding link=xoap in the url string $url = "http://xoap.weather.com/weather/local/".$id."?link=xoap&prod=xoap&par=".$this->_partnerID."&key=".$this->_licenseKey; Also to be mentionned: Forecasts are now only available for the 5 days against 10 days before.
 [2008-05-10 15:43 UTC] eru (Alexander Wirtz)
This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.
 [2008-09-30 10:53 UTC] geeks2rent (Conny Dittmann)
Hello All, I am using Clark Connect and horde and this is my 3 day fiddeling with weather.com. as soon as it is selected it destroys the login portal of the user inside horde. i found a lot of bodging and work around's so far none was working for me. i saw many got the problem - but no real solution given. not on how to disable this function in horde so user cant select it on the first place. i am ever so happy to supply logs, configs, "pear list" , etc. if required. could please someone point me to the right direction.