Skip to content
This repository has been archived by the owner on May 12, 2020. It is now read-only.

Security error at zorbuth #74

Open
smitgd opened this issue Jul 16, 2016 · 2 comments
Open

Security error at zorbuth #74

smitgd opened this issue Jul 16, 2016 · 2 comments

Comments

@smitgd
Copy link
Contributor

smitgd commented Jul 16, 2016

When I use the link provided to obtain a woeid I can't get the woeid but only the message "Security Error!". Not sure if this is just a temporary error or something really broken or changed at the zorbuth site (maybe they now have a different url that uses https, but if I prepend https to the current url I only get 404).
If I do a google search on "woeid lookup" I find many other sites that successfully do the lookup.
I would recommend adding a note to the configuration page that a web search on "woeid" can also find other sites that do the lookup if or when the zorbuth site fails.
smitgd added a commit to smitgd/com.librehat.yahooweather that referenced this issue Jul 17, 2016
@smitgd
Re librehat#74: Security error at zorbuth
7316b4a 
This adds support for a location string that overrides the WOEID
entry. So now can enter a free form location such as "Pine Crest
TN" or "London, england". Only when location is left blank is the
entered WOEID used in the yahoo query. Otherwise, the location
string (that can also be a zipcode) is used in the query.
Also, added note that a web search can be done to find alternate
sites to obtain WOEID if needed, plus some clean up on configuration
labels (periods at end of sentences, etc.).
	modified:   plasmoid/contents/config/main.xml
	modified:   plasmoid/contents/ui/ConfigGeneral.qml
	modified:   plasmoid/contents/ui/Yahoo.qml
@smitgd
Copy link
Contributor Author

smitgd commented Jul 17, 2016 •
edited
Loading

Looking closer at this, it appears that the WOEID is not actually required. There is a supported yahoo query that allows a location string or zip code to be used instead of WOEID. It is illustrated here: https://developer.yahoo.com/weather/
So I am making a pull req. that supports either a location string or a WOEID entered on the general configuration page. If the location (now first on the page) is left blank, the WOEID entry is used. But if location string is not empty, it is used and the WOEID entry is ignored. This will not break existing configurations since the default location string is blank so the stored WOEID will be used as before.

@smitgd
Copy link
Contributor Author

smitgd commented Jul 26, 2016

FYI, zourbuth site is now working again for me. Not currently seeing "Security Error" when looking up woeid. (See PR #75 for changes related to this issue.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@smitgd