Re: [Ipodder-betatesters] Need better Basic Auth Error Handling

[Andrew G. <aeg...@mi...>]

Thanks for the suggestions, Paul!  I like your thinking.

I think we ought to at least be able to squeeze in the error message.

The pop-up window ought to be do-able, though we'll have to put in some 
kind of check to disable the popup during middle-of-the-night scheduled 
runs, so it doesn't bring everything to a grinding halt if the user is 
away from their machine.

Andrew


Whelan, Paul A. (P&G Account Team) wrote:

 > Wanted to see what kind of notification a user would get if they 
entered a bad name and password for a feed (or if they would get 
anything at all): see below details.  I think there needs to be a more 
explicit error message when authentication fails.  People's passwords 
will change from time to time and their feeds will stop working quietly 
in the background.
 > A second suggestion on this front is to have ipodder a request to 
enter a password pop-up on the screen when a basic auth request is 
encountered by ipodder or if the basic auth information supplied by user 
fails.  See RSSOwl behaviour in this matter for an example.  It prompts 
the user for a user name and password at the time of the feed (if not 
supplied when the feed is configured), and also asks the user whether 
they want to store it and keep using it.  If they answer yes, the feed 
configuration is updated.  This insures that feeds requiring basic auth 
keep working reliably.
 >
 > Paul
 >
 >
 > Added feed and entered good name/bad password Got the following in 
the log
 >
 > BasicGrabber 31130736 u'http://idea-srv001/rssfeed/rssfeed.xml' 
reports: This traceback is new to us:
 > Traceback (most recent call last):
 > File "ipodder\grabbers.pyc", line 585, in _translateUsualDownloadErrors
 > File "ipodder\grabbers.pyc", line 422, in _open_resource
 > File "urllib2.pyc", line 326, in open
 > File "urllib2.pyc", line 306, in _call_chain
 > File "urllib2.pyc", line 901, in http_open
 > File "urllib2.pyc", line 895, in do_open
 > File "urllib2.pyc", line 346, in error
 > File "urllib2.pyc", line 306, in _call_chain
 > File "urllib2.pyc", line 834, in http_error_401
 > File "urllib2.pyc", line 725, in http_error_auth_reqed
 > ValueError: AbstractDigestAuthHandler doesn't know about Basic
 > FeedScanningJob 31130640 'Remote Test Channel' reports: Can't grab 
Remote Test Channel: unexpected error <exceptions.ValueError instance at 
0x01DA0D00>
 >
 >
 > Went back and looked at feed...hmmm...name and password field 
empty.... entered same good name/bad password again, got the following 
in the log:
 >
 >
 > BasicGrabber 31130704 u'http://idea-srv001/rssfeed/rssfeed.xml' 
reports: This traceback is new to us:
 > Traceback (most recent call last):
 > File "ipodder\grabbers.pyc", line 585, in _translateUsualDownloadErrors
 > File "ipodder\grabbers.pyc", line 422, in _open_resource
 > File "urllib2.pyc", line 326, in open
 > File "urllib2.pyc", line 306, in _call_chain
 > File "urllib2.pyc", line 901, in http_open
 > File "urllib2.pyc", line 895, in do_open
 > File "urllib2.pyc", line 346, in error
 > File "urllib2.pyc", line 306, in _call_chain
 > File "urllib2.pyc", line 659, in http_error_401
 > File "urllib2.pyc", line 638, in http_error_auth_reqed
 > File "urllib2.pyc", line 648, in retry_http_basic_auth
 > File "urllib2.pyc", line 326, in open
 > File "urllib2.pyc", line 306, in _call_chain
 > File "urllib2.pyc", line 901, in http_open
 > File "urllib2.pyc", line 895, in do_open
 > File "urllib2.pyc", line 346, in error
 > File "urllib2.pyc", line 306, in _call_chain
 > File "urllib2.pyc", line 834, in http_error_401
 > File "urllib2.pyc", line 725, in http_error_auth_reqed
 > ValueError: AbstractDigestAuthHandler doesn't know about Basic
 > FeedScanningJob 31130608 'Remote Test Channel' reports: Can't grab 
Remote Test Channel: unexpected error <exceptions.ValueError instance at 
0x01DB6468>
 >
 > Edited feed and fixed password.  Feed worked as designed
 >
 >
 >