| Issue 32: | Self-explaining error messages | |
| 3 people starred this issue and may be notified of changes. | Back to list |
I've just seen that the error messages could be improved: "Invalid request." (line 201) - You get this message, if the host you've entered (e.g. http://martin-thoma.blogger.com) doesn't exist. So the message could be "Invalid request. Host 'http://martin-thoma.blogger.com' doesn't exist.
Apr 10, 2011
Well, I would understand these error messages better. I hope I understood the error messages correct: l. 79: 'You must have either https wrappers or curl enabled.' -> 'You must have either https wrappers or cURL enabled. cURL is a command line tool for transferring files.' (what are https wrappers?) l. 201: 'Invalid request.' -> 'Invalid request. Host "http://martin-thoma.blogger.com" doesn't exist.' l. 463: 'No servers found!' -> 'No OpenID-Server found at "http://blogger.com". Perhaps you have misspelled your OpenID?' l. 465: 'Endless redirection!' How can this happen?
Apr 11, 2011
Ad l. 79: I expect that people who don't know about something in php itself will search for it in the docs. For example: http://php.net/wrappers http://php.net/curl (also, while curl is a very useful command-line tool, it's also a library -- and that's how it's used in LightOpenID) As I don't explain what is a OpenID (and some people using the library *don't* know that), I don't see a point in explaining every configuration detail of php. Especially since most default configurations already have both enabled, and if you have neither, you usually can't do anything about it. Ad l. 201: Changed to "Could not connect to $url.", Although the $url can be any url in the path. Ad l. 463: Will be the first sentence of your suggestion. Ad l. 465: If you visit a url, let's say `http://example.com/openid`, and it has a header "Location: /openid", then LightOpenId follows to `http://example.com/openid` again. This way, a cycle is created and it would cause an infinite loop. That's why (as Yadis specification suggests) I throw an exception after 5 redirects.
Apr 21, 2011
(No comment was entered for this change.)
Status:
Fixed
|
Labels: -Type-Defect Type-Enhancement