Modify

Opened 10 years ago

Closed 9 years ago

#1484 closed enhancement (fixed)

[PATCH] Chained Password Stores

Reported by: John Hampton Owned by: John Hampton
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 John Hampton 10 years ago.
0.10.acct_mgr.api.py.diff (6.0 KB) - added by John Hampton 10 years ago.
0.10.acct_mgr.db.py.diff (952 bytes) - added by John Hampton 10 years ago.
0.10.acct_mgr.templates.admin_accountsconfig.cs.diff (950 bytes) - added by John Hampton 10 years ago.
0.10.acct_mgr.templates.admin_users.cs.diff (1.2 KB) - added by John Hampton 10 years ago.
0.10.acct_mgr.web_ui.py.diff (2.8 KB) - added by John Hampton 10 years ago.
0.11.acct_mgr.admin.py.diff (6.5 KB) - added by John Hampton 10 years ago.
0.11.acct_mgr.api.py.diff (5.8 KB) - added by John Hampton 10 years ago.
0.11.acct_mgr.db.py.diff (952 bytes) - added by John Hampton 10 years ago.
0.11.acct_mgr.templates.admin_accountsconfig.html.diff (808 bytes) - added by John Hampton 10 years ago.
0.11.acct_mgr.templates.admin_users.html.diff (1.3 KB) - added by John Hampton 10 years ago.
0.11.acct_mgr.templates.register.html.diff (1.3 KB) - added by John Hampton 10 years ago.
0.11.acct_mgr.web_ui.py.diff (3.4 KB) - added by John Hampton 10 years ago.
0.10.mega_patch.diff (17.6 KB) - added by John Hampton 10 years ago.
0.10 patch rollup
0.11.mega_patch.diff (20.1 KB) - added by John Hampton 10 years ago.
0.11 patch rollup
0.10.acct_mgr.api.py.2.diff (157 bytes) - added by anonymous 9 years ago.
0.11.mega_rollup.diff (20.2 KB) - added by John Hampton 9 years ago.
Fixed patch against latest trunk of acct_mgr

Download all attachments as: .zip

Change History (22)

Changed 10 years ago by John Hampton

Attachment: 0.10.acct_mgr.admin.py.diff added

Changed 10 years ago by John Hampton

Attachment: 0.10.acct_mgr.api.py.diff added

Changed 10 years ago by John Hampton

Attachment: 0.10.acct_mgr.db.py.diff added

Changed 10 years ago by John Hampton

Changed 10 years ago by John Hampton

Changed 10 years ago by John Hampton

Changed 10 years ago by John Hampton

Attachment: 0.11.acct_mgr.admin.py.diff added

Changed 10 years ago by John Hampton

Attachment: 0.11.acct_mgr.api.py.diff added

Changed 10 years ago by John Hampton

Attachment: 0.11.acct_mgr.db.py.diff added

Changed 10 years ago by John Hampton

Changed 10 years ago by John Hampton

Changed 10 years ago by John Hampton

Changed 10 years ago by John Hampton

Attachment: 0.10.mega_patch.diff added

0.10 patch rollup

Changed 10 years ago by John Hampton

Attachment: 0.11.mega_patch.diff added

0.11 patch rollup

comment:1 Changed 10 years ago by John Hampton

Matt,

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

comment:2 Changed 10 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 9 years ago by anonymous

Attachment: 0.10.acct_mgr.api.py.2.diff added

comment:3 Changed 9 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 9 years ago by John Hampton

Attachment: 0.11.mega_rollup.diff added

Fixed patch against latest trunk of acct_mgr

comment:4 Changed 9 years ago by John Hampton

Owner: changed from Matt Good to John Hampton
Status: newassigned
Trac Release: 0.100.11

comment:5 Changed 9 years ago by John Hampton

Resolution: fixed
Status: assignedclosed

Feature added in [4638]

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain John Hampton.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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

 
Note: See TracTickets for help on using tickets.