Modify

Opened 16 months ago

Last modified 16 months ago

#11303 new defect

[Patch] req.remote_user is not always set

Reported by: korn Owned by: clp
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)

accountldap_english_authname.diff (8.2 KB) - added by korn 16 months ago.

Download all attachments as: .zip

Change History (2)

comment:1 Changed 16 months ago by korn

  • Summary changed from req.remote_user is not always set to [Patch] req.remote_user is not always set

Changed 16 months ago by korn

Add Comment

Modify Ticket

Action
as new The owner will remain clp.
Author


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

 
Note: See TracTickets for help on using tickets.