On Wed, 16 Jul 2003, Matthew Schumacher wrote:

> Alan,
>
> I don't know if this is something you want to include but I thought I
> would mail it to you in case you do want it.
>
> This patch causes the rlm_ldap module to reject usernames that have
> whitespace in them.  This was causing me a lot of grief because ldap was
> accepting usernames with whitespace causing the NAS to send accounting
> messages with white space.  This white space caused the Simultaneous-Use
> code to break and it also caused a lot of accounting headaches.
>
> There was also one case there a user was authenticated with more than 30
> chars of whitespace which broke accounting when the rlm_postgres module
> when to put that username in a varchar(20) column.
>
> BTW, I can barely hack my way around c so hopefully there isn't anything
> wrong with my code.
>
> schu
>

1. You can do the same thing with a simple regex in the users file

2. There is also the nospace_user directive in radiusd.conf

# nospace_user / nospace_pass:
#
#  Some users like to enter spaces in their username or password
#  incorrectly.  To save yourself the tech support call, you can
#  eliminate those spaces here:
#
# Default is 'no' (don't remove spaces)
# Valid values = "before" / "after" / "no" (explanation above)
#
nospace_user = no
nospace_pass = no


So in general I don't think there's any need for this patch...

--
Kostas Kalevras         Network Operations Center
[EMAIL PROTECTED]       National Technical University of Athens, Greece
Work Phone:             +30 210 7721861
'Go back to the shadow' Gandalf

- 
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to