Modify

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#6329 closed defect (duplicate)

Login with SessionStore not working after upgrade to Trac 0.11.6

Reported by: klaus@… Owned by: mgood
Priority: high Component: AccountManagerPlugin
Severity: major Keywords: SessionStore, 0.11.6, LogIn
Cc: Trac Release: 0.11

Description

Hi,

for the WebsiteBaker TRAC installation we have your AccountManagerPlugin installed sind I took over the management (project.websitebaker2.org) of this server.

Yesterday I upgraded the TRAC Version to 0.11.6 and at first glance had no issue, everything seemd fine. I logged off (the session was still OK and tried to log back in this morning and get following error:

Traceback (most recent call last):
  File "/usr/lib/python2.4/site-packages/Trac-0.11.6-py2.4.egg/trac/web/api.py", line 377, in send_error
    'text/html')
  File "/usr/lib/python2.4/site-packages/Trac-0.11.6-py2.4.egg/trac/web/chrome.py", line 733, in render_template
    message = req.session.pop('chrome.%s.%d' % (type_, i))
  File "/usr/lib/python2.4/site-packages/Trac-0.11.6-py2.4.egg/trac/web/api.py", line 195, in __getattr__
    value = self.callbacks[name](self)
  File "/usr/lib/python2.4/site-packages/Trac-0.11.6-py2.4.egg/trac/web/main.py", line 265, in _get_session
    return Session(self.env, req)
  File "/usr/lib/python2.4/site-packages/Trac-0.11.6-py2.4.egg/trac/web/session.py", line 149, in __init__
    if req.authname == 'anonymous':
  File "/usr/lib/python2.4/site-packages/Trac-0.11.6-py2.4.egg/trac/web/api.py", line 195, in __getattr__
    value = self.callbacks[name](self)
  File "/usr/lib/python2.4/site-packages/Trac-0.11.6-py2.4.egg/trac/web/main.py", line 134, in authenticate
    authname = authenticator.authenticate(req)
  File "build/bdist.linux-x86_64/egg/acct_mgr/web_ui.py", line 430, in wrap
  File "build/bdist.linux-x86_64/egg/acct_mgr/web_ui.py", line 440, in authenticate
  File "build/bdist.linux-x86_64/egg/acct_mgr/web_ui.py", line 467, in _remote_user
  File "build/bdist.linux-x86_64/egg/acct_mgr/api.py", line 140, in check_password
  File "build/bdist.linux-x86_64/egg/acct_mgr/db.py", line 76, in check_password
  File "build/bdist.linux-x86_64/egg/acct_mgr/pwhash.py", line 52, in check_hash
  File "build/bdist.linux-x86_64/egg/acct_mgr/pwhash.py", line 48, in generate_hash
  File "build/bdist.linux-x86_64/egg/acct_mgr/pwhash.py", line 56, in _encode
AttributeError: 'NoneType' object has no attribute 'encode'

First I uninstalled and reinstalled the Version for the 0.11 release of your plugin to no avail. Then I installed the latest release (TracAccountManager-0.2.1dev_r7165-py2.4.egg) which is meant for the 0.12 release of TRAC, but I figured it wouldn't matter, worth trying. No change to current status.

From what I can figure, I am missing something in the configuration, which also makes me wonder why it worked before. Anyway, here is part of my trac.ini

[account-manager]
account_changes_notify_addresses = klaus@weitzel.info
force_passwd_change = false
hash_method = HtDigestHashMethod
notify_actions = new
password_file = /home/wb/conf/.htpasswd
password_store = SessionStore

[components]
acct_mgr.* = enabled
acct_mgr.web_ui.EmailVerificationModule = disabled
acct_mgr.htfile.HtDigestStore = disabled
acct_mgr.htfile.HtPasswdStore = disabled
acct_mgr.http.HttpAuthStore = disabled
acct_mgr.db.SessionStore = enabled
acct_mgr.pwhash.htpasswdhashmethod = disabled
acct_mgr.pwhash.htdigesthashmethod = enabled
acct_mgr.svnserve.* = disabled
acct_mgr.admin.AccountManagerAdminPage = enabled
acct_mgr.web_ui.AccountModule = enabled
acct_mgr.web_ui.RegistrationModule = disabled
acct_mgr.web_ui.EmailVerificationModule = disabled
ticketdelete.* = enabled
trac.web.auth.loginmodule = disabled
tracnav.tracnav.tracnav = enabled
tracspamfilter.* = enabled
tracusermanager.* = enabled

I hope this is enough to help me out ;-)

cheers

Klaus

Attachments (0)

Change History (7)

comment:1 Changed 5 years ago by anonymous

Hi there,

i had the same problem with 11.6 when adding a new user. the traces show that acct_mgr.pwhash.HtDigestHashMethod.realm is not set.

this worked for me:

; the name of the authentication "realm"
; it can be any text to identify your site or project
htdigest_realm = TracRealm

setting htdigest_realm in the trac.ini solved the problem.

kind regards

Marc

comment:2 Changed 5 years ago by anonymous

Same problem here. setting htdigest_realm solved the problem (but had to reset the password afterwards), thanks this saved my day.

comment:3 Changed 5 years ago by anonymous

Hi, setting htdigest_realm = solves the problem too, without having to change or reset passwords

comment:4 Changed 5 years ago by hoff.st@…

a random 'htdigest_realm' entry saved my day too, many thanks to anonymous alias Marc[[BR]] password reset required for all users here too

(first I needed an new account with TRAC_ADMIN permission to reset password of my current trac-admin account - not a small pain)

now thinking, how to do upcoming migration on production system this test was made for ...

tested here with

trac 0.12dev-r9115 acct_mgr 0.2.1dev_r7165-py2.5.egg

comment:5 Changed 5 years ago by klaus@…

Hi Guys,

unfortunatly this does not seem to work for our environment, at least not with the session store DB

acct_mgr.db.SessionStore = enabled

Currently we am using the htdigest sessionstore

acct_mgr.htdigest_realm = TracRealm
acct_mgr.htfile.htdigeststore = enabled
acct_mgr.htfile.htdigeststore = enabled

What I wonder now, is if we need to "change" the ini setting to

acct_mgr.htfile.htdigest_realm = TracRealm
acct_mgr.htfile.htdigeststore = enabled

to make it work with the sessions tore, or to change it completly to

acct_mgr.db.htdigest_realm = TracRealm

Further help is apreciated ...

cheers

Klaus

comment:6 follow-up: Changed 5 years ago by pacopablo

  • Resolution set to duplicate
  • Status changed from new to closed

This is a duplicate of #6453 (or that is a dup of this, whatever). You can find the solution there. I will commit a patch in the next day or two.

comment:7 in reply to: ↑ 6 Changed 5 years ago by anonymous

Replying to pacopablo:

This is a duplicate of #6453 (or that is a dup of this, whatever). You can find the solution there. I will commit a patch in the next day or two.

Would have been more sensible to close the newer ticket as dup. But I see, that you posted relevant information there, probably before recognizing the older dup here. The proposed simple solution is - of course - highly appreciated. Thanks.

Add Comment

Modify Ticket

Action
as closed The owner will remain mgood.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.