directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Emmanuel Lecharny <elecha...@gmail.com>
Subject Re: svn commit: r1144962 - /directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java
Date Mon, 11 Jul 2011 06:55:43 GMT
I'm not sure it"s a good idea to setup a default session, at least to 
admin. If we consider the normal (ie, not embedded) server, we don't set 
any session, the default session is Anonymous (of course if allowed). 
IMO, this might be a security breach too.

What was the rational for this modificatioon, Alex ?

On 7/10/11 11:23 PM, akarasulu@apache.org wrote:
> Author: akarasulu
> Date: Sun Jul 10 21:23:02 2011
> New Revision: 1144962
>
> URL: http://svn.apache.org/viewvc?rev=1144962&view=rev
> Log:
> session member should be reset to an admin session when the directory service is set
>
> Modified:
>      directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java
>
> Modified: directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java
> URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java?rev=1144962&r1=1144961&r2=1144962&view=diff
> ==============================================================================
> --- directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java
(original)
> +++ directory/apacheds/trunk/core-api/src/main/java/org/apache/directory/server/core/LdapCoreSessionConnection.java
Sun Jul 10 21:23:02 2011
> @@ -1286,5 +1286,6 @@ public class LdapCoreSessionConnection i
>       {
>           this.directoryService = directoryService;
>           this.schemaManager = directoryService.getSchemaManager();
> +        this.session = directoryService.getAdminSession();
>       }
>   }
>
>
>


-- 
Regards,
Cordialement,
Emmanuel L├ęcharny
www.iktek.com


Mime
View raw message