Opened 16 years ago
Closed 15 years ago
#8107 closed enhancement (fixed)
a SMTPAuthenticationError shouldn't be presented as an internal error
Reported by: | Owned by: | Remy Blank | |
---|---|---|---|
Priority: | normal | Milestone: | 0.12 |
Component: | notification | Version: | 0.11.2.1 |
Severity: | minor | Keywords: | |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
How to Reproduce
While doing a GET operation on /wiki
, Trac issued an internal error.
(please provide additional details here)
User Agent was: Mozilla/5.0 (Windows; U; Windows NT 5.1; pt-BR; rv:1.9.0.6) Gecko/2009011913 Firefox/3.0.6
System Information
Trac | 0.11.2.1
|
Python | 2.5.2 (r252:60911, Jul 31 2008, 17:28:52) [GCC 4.2.3 (Ubuntu 4.2.3-2ubuntu7)]
|
setuptools | 0.6c9
|
psycopg2 | 2.0.8
|
Genshi | 0.5.1
|
Subversion | 1.4.6 (r28521)
|
jQuery: | 1.2.6
|
Python Traceback
Traceback (most recent call last): File "/usr/lib/python2.5/site-packages/Trac-0.11.2.1-py2.5.egg/trac/web/main.py", line 432, in _dispatch_request dispatcher.dispatch(req) File "/usr/lib/python2.5/site-packages/Trac-0.11.2.1-py2.5.egg/trac/web/main.py", line 216, in dispatch self._post_process_request(req, *resp) File "/usr/lib/python2.5/site-packages/Trac-0.11.2.1-py2.5.egg/trac/web/main.py", line 308, in _post_process_request resp = f.post_process_request(req, *resp) File "build/bdist.linux-i686/egg/acct_mgr/web_ui.py", line 560, in post_process_request self._send_email(req) File "build/bdist.linux-i686/egg/acct_mgr/web_ui.py", line 599, in _send_email notifier.notify(req.authname, req.session['email_verification_token']) File "build/bdist.linux-i686/egg/acct_mgr/web_ui.py", line 529, in notify SingleUserNofification.notify(self, username, subject) File "build/bdist.linux-i686/egg/acct_mgr/web_ui.py", line 123, in notify NotifyEmail.notify(self, username, subject) File "/usr/lib/python2.5/site-packages/Trac-0.11.2.1-py2.5.egg/trac/notification.py", line 246, in notify Notify.notify(self, resid) File "/usr/lib/python2.5/site-packages/Trac-0.11.2.1-py2.5.egg/trac/notification.py", line 134, in notify self.begin_send() File "/usr/lib/python2.5/site-packages/Trac-0.11.2.1-py2.5.egg/trac/notification.py", line 328, in begin_send self.server.login(self.user_name, self.password) File "/usr/lib/python2.5/smtplib.py", line 591, in login raise SMTPAuthenticationError(code, resp) SMTPAuthenticationError: (535, '5.7.1 Username and Password not accepted. Learn more at\n5.7.1 http://mail.google.com/support/bin/answer.py?answer=14257 z20sm4589536pod.12')
Attachments (0)
Change History (4)
comment:1 by , 16 years ago
comment:2 by , 16 years ago
Component: | general → notification |
---|---|
Milestone: | → 0.12 |
Owner: | set to |
Severity: | normal → minor |
Summary: | SMTPAuthenticationError: (535, '5.7.1 Username and Password not accepted. Learn more at\n5.7.1 http://mail.google.com/support/bin/answer.py?answer=14257 z20sm4589536pod.12') → a SMTPAuthenticationError shouldn't be presented as an internal error |
Type: | defect → enhancement |
Still, we shouldn't end up with a backtrace in this case, I think.
Catching SMTPAuthenticationError and throwing a nicer Trac error would be better in this case.
comment:3 by , 15 years ago
Owner: | changed from | to
---|
I'll add a nicer message, but maybe as a warning message rather than a TracError
.
comment:4 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
With [8739], a warning is displayed if an error occurs while sending notifications.
Did you tried what is written at the given link? http://mail.google.com/support/bin/answer.py?answer=14257
I did it and it worked so far on my testsystem. I only had to do the captcha test and rerun the whole thing.
cheers, gregor