culturecommunication / ginco

Gestion Informatisée de Nomenclatures Collaboratives et Ouvertes
http://culturecommunication.github.io/ginco/
Other
41 stars 23 forks source link

Les paramètres de connexion LDAP sont hardcodés dans le fichier de config Spring #32

Open tfrancart opened 9 years ago

tfrancart commented 9 years ago

Bonjour

Les paramètres de connexion au LDAP sont en partie hardcodés dans le fichier de config Spring ginco-admin/src/main/resources/spring/applicationContext-security.xml :

<ldap-server url="${ldap.base.provider.url}/o=gouv,c=fr"
                 manager-dn="${ldap.security.principal}" manager-password="${ldap.security.credentials}"/>
    <authentication-manager alias="authenticationManager" erase-credentials="false">
        <ldap-authentication-provider
                user-search-base="ou=personnes" user-search-filter="uid={0}"
                user-context-mapper-ref="gincoUserDetailsContextMapper"/>
        <authentication-provider>
            <user-service>
                <user name="${default.user.login}" authorities="ROLE_ADMIN" disabled="false"
                      password="${default.user.password}"/>
                <user name="${default.expert.login}" authorities="ROLE_USER" disabled="false"
                      password="${default.expert.password}"/>
                <user name="${default.responsable.login}" authorities="ROLE_USER" disabled="false"
                      password="${default.responsable.password}"/>
            </user-service>
        </authentication-provider>
    </authentication-manager>

Ces éléments de config (o=gouv,c=fr, ou=personnes, uid={0}) devraient être lu depuis le fichier ginco.properties. Cela empêche l'utilisation de GINCO par la communauté des potentiels réutilisateurs.

hufon commented 9 years ago

Cette modification est très simple. Rien ne vous empêche de faire une pull request.

tfrancart commented 9 years ago

Entendu, c'est ce que j'ai fait. Merci.