[Mod_edir-dev] r134 - trunk/mod_edir

[Guenter Knauf]

Hi Pat,
I was told by users that your commit below broke aliased user lookup at all now with mod_edir 1.0.13;
also its unclear to me how the removal of the code should archive what you state in the log...?

In addition we have another issue with mod_edir that connections again stay for ever open once a user has logged in through mod_edir - but this might be a libc prob again..... - you you re-create that?
It leads to the situation that the LDAP server goes up to 100% usage, and becomes unavailable....

Juergen, can you please re-post your last mail with the LDAP status here again in english?
ans also please your current setup so that Pat is able to re-create?
thanks!

Guenter.

> Author: patricklcam
> Date: 2007-03-13 12:20:09 -0600 (Tue, 13 Mar 2007)
> New Revision: 134

> Modified:
>    trunk/mod_edir/rdirutils.c
> Log:
> Changed  edir_resolveLDAPalias() to return the FDN of the aliased user
> if found NULL otherwise. (pl )


> Modified: trunk/mod_edir/rdirutils.c
> ===================================================================
> --- trunk/mod_edir/rdirutils.c	2007-03-13 18:18:45 UTC (rev 133)
> +++ trunk/mod_edir/rdirutils.c	2007-03-13 18:20:09 UTC (rev 134)
> @@ -268,7 +268,7 @@

>  /* edir_resolveLDAPalias()
>  **
> -** Returns - pointer to aliased user name or NULL of an alias is not
> found
> +** Returns - FDN pointer to aliased user name or NULL if an alias is not
> found
>  */
>  char *edir_resolveLDAPalias(request_rec *r, LDAP *ld, char *searchBase,
>  char *username)
>  {
> @@ -337,23 +337,6 @@
>              }
>              ber_free(ber, 0);
>          }
> -
> -        /* If we found an alias DN then pull out the real user name and
> -           replace the alias user name */
> -        if (rawDN) {
> -            char *dend;
> -
> -            rawDN = strchr(rawDN, '=');
> -            if (rawDN) {
> -                rawDN++;
> -                dend = strchr (rawDN, ',');
> -                if (dend) {
> -                    *dend = '\0';
> -                }
> -                ap_log_rerror(APLOG_MARK, APLOG_NOERRNO|APLOG_DEBUG, 0,
> r,
> -                    "Replacing alias user name %s with %s", username,
> rawDN);
> -            }
> -        }
>      }
>      ldap_msgfree( searchResult );


> _______________________________________________
> Mod_edir-dev mailing list
> Mod_edir-dev at forge.novell.com
> http://forge.novell.com/mailman/listinfo/mod_edir-dev

best regards,

Guenter Knauf
Novell CNE, NetWare Developer
__________________________________________________________________

GK Netzwerktechnik
http://www.gknw.de/
__________________________________________________________________