ipodder-betatesters

[Ipodder-betatesters] Need better Basic Auth Error Handling

[Whelan, P. A. (P&G A. Team) <pw...@hp...>]

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.  =20
=20
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.
=20
Paul
=20
=20
Added feed and entered good name/bad password Got the following in the
log
=20
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>
=20
=20
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:
=20
=20
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>
=20
Edited feed and fixed password.  Feed worked as designed
=20
=20
=20

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
 >
 >
 >