Modify ↓
Opened 11 years ago
Last modified 3 years ago
#11303 new defect
[Patch] req.remote_user is not always set
Reported by: | Andras Korn | Owned by: | Carlos López Pérez |
---|---|---|---|
Priority: | normal | Component: | AccountLdapPlugin |
Severity: | normal | Keywords: | |
Cc: | Trac Release: |
Description
Hi,
I played around with this plugin a bit because it didn't seem to do anything.
Apparently, req.remote_user
isn't (always?) set, so I modified it to use req.authname
instead. It now works for me, but I don't know if this is a correct solution.
The attached patch contains this change as well as:
- messages and HTML template (of 0.12 branch) translated into English (sorry, not proper l10n, I just updated the hardcoded strings);
- it retrieves displayName instead of cn to get the name (this should really be configurable, I suppose);
- it now supports non-ascii characters in names;
- some code typos were fixed;
- I added some debug logging.
Attachments (1)
Change History (3)
comment:1 Changed 11 years ago by
Summary: | req.remote_user is not always set → [Patch] req.remote_user is not always set |
---|
Changed 11 years ago by
Attachment: | accountldap_english_authname.diff added |
---|
comment:2 Changed 3 years ago by
Note: See
TracTickets for help on using
tickets.
See also #3171.