Modify

Opened 7 years ago

Closed 6 years ago

#1484 closed enhancement (fixed)

[PATCH] Chained Password Stores

Reported by: pacopablo Owned by: pacopablo
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

Currently, only one implementation of IPasswordStore can be used for authentication. This means that you can use htpasswd OR db auth OR LDAP, etc. However, for some setups, especially those that use LDAP (specifically of the hyperActive Directory kind), it is helpful to have multiple authorization sources. This would allow for a company to have their main users in AD while using htpasswd or db auth for outside users/consultants.

The attached patches make it so that the password_store option becomes an OrderedExtensionsOption. This allows the user to specify the password stores that they want to use. For backwards compatibility, simple specify only one IPasswordStore implementation.

The patch also modifies the admin interface. Instead of radio buttons, select boxes are used so that the password stores can be ordered via webadmin

Finally, small patch to user creation. If the creation fails for some reason, such as passwords don't match, it fills in the dialog with the previously entered information.

Would love to see this get merged ;)

Attachments (17)

0.10.acct_mgr.admin.py.diff (5.7 KB) - added by pacopablo 7 years ago.
0.10.acct_mgr.api.py.diff (6.0 KB) - added by pacopablo 7 years ago.
0.10.acct_mgr.db.py.diff (952 bytes) - added by pacopablo 7 years ago.
0.10.acct_mgr.templates.admin_accountsconfig.cs.diff (950 bytes) - added by pacopablo 7 years ago.
0.10.acct_mgr.templates.admin_users.cs.diff (1.2 KB) - added by pacopablo 7 years ago.
0.10.acct_mgr.web_ui.py.diff (2.8 KB) - added by pacopablo 7 years ago.
0.11.acct_mgr.admin.py.diff (6.5 KB) - added by pacopablo 7 years ago.
0.11.acct_mgr.api.py.diff (5.8 KB) - added by pacopablo 7 years ago.
0.11.acct_mgr.db.py.diff (952 bytes) - added by pacopablo 7 years ago.
0.11.acct_mgr.templates.admin_accountsconfig.html.diff (808 bytes) - added by pacopablo 7 years ago.
0.11.acct_mgr.templates.admin_users.html.diff (1.3 KB) - added by pacopablo 7 years ago.
0.11.acct_mgr.templates.register.html.diff (1.3 KB) - added by pacopablo 7 years ago.
0.11.acct_mgr.web_ui.py.diff (3.4 KB) - added by pacopablo 7 years ago.
0.10.mega_patch.diff (17.6 KB) - added by pacopablo 7 years ago.
0.10 patch rollup
0.11.mega_patch.diff (20.1 KB) - added by pacopablo 7 years ago.
0.11 patch rollup
0.10.acct_mgr.api.py.2.diff (157 bytes) - added by anonymous 7 years ago.
0.11.mega_rollup.diff (20.2 KB) - added by pacopablo 6 years ago.
Fixed patch against latest trunk of acct_mgr

Download all attachments as: .zip

Change History (22)

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

Changed 7 years ago by pacopablo

0.10 patch rollup

Changed 7 years ago by pacopablo

0.11 patch rollup

comment:1 Changed 7 years ago by pacopablo

Matt,

Wanted to see if you've had the opportunity to take a look at the patches?

comment:2 Changed 7 years ago by thomasvs

I tried the 0.10 rollup of this patch and it works great for me. I'd like to see this merged too.

Changed 7 years ago by anonymous

comment:3 Changed 7 years ago by anonymous

TracLDAPAuth does not support adding/listing/removing users, so I added some attribute checks to api.py to precent errors.

Changed 6 years ago by pacopablo

Fixed patch against latest trunk of acct_mgr

comment:4 Changed 6 years ago by pacopablo

  • Owner changed from mgood to pacopablo
  • Status changed from new to assigned
  • Trac Release changed from 0.10 to 0.11

comment:5 Changed 6 years ago by pacopablo

  • Resolution set to fixed
  • Status changed from assigned to closed

Feature added in [4638]

Add Comment

Modify Ticket

Action
as closed .
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.