Opened 18 years ago
Closed 16 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)
Change History (22)
Changed 18 years ago by
| Attachment: | 0.10.acct_mgr.admin.py.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.10.acct_mgr.api.py.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.10.acct_mgr.db.py.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.10.acct_mgr.templates.admin_accountsconfig.cs.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.10.acct_mgr.templates.admin_users.cs.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.10.acct_mgr.web_ui.py.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.11.acct_mgr.admin.py.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.11.acct_mgr.api.py.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.11.acct_mgr.db.py.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.11.acct_mgr.templates.admin_accountsconfig.html.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.11.acct_mgr.templates.admin_users.html.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.11.acct_mgr.templates.register.html.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.11.acct_mgr.web_ui.py.diff added |
|---|
Changed 18 years ago by
| Attachment: | 0.10.mega_patch.diff added |
|---|
comment:1 Changed 17 years ago by
Matt,
Wanted to see if you've had the opportunity to take a look at the patches?
comment:2 Changed 17 years ago by
I tried the 0.10 rollup of this patch and it works great for me. I'd like to see this merged too.
Changed 17 years ago by
| Attachment: | 0.10.acct_mgr.api.py.2.diff added |
|---|
comment:3 Changed 17 years ago by
TracLDAPAuth does not support adding/listing/removing users, so I added some attribute checks to api.py to precent errors.
Changed 16 years ago by
| Attachment: | 0.11.mega_rollup.diff added |
|---|
Fixed patch against latest trunk of acct_mgr
comment:4 Changed 16 years ago by
| Owner: | changed from Matt Good to John Hampton |
|---|---|
| Status: | new → assigned |
| Trac Release: | 0.10 → 0.11 |
comment:5 Changed 16 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Feature added in [4638]
0.10 patch rollup