Skip to content

Commit

Permalink
Removed based64 encryption mechanism for ldap manager password (#105)
Browse files Browse the repository at this point in the history
  • Loading branch information
xdurvak authored Jun 12, 2018
1 parent 1b72f0e commit 07e649c
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 9 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
## 0.10.0
- Removed based64 encryption mechanism for ldap manager password

## 0.9.9
- - Added jasypt-spring-boot-starter dependency to support open text encryption

Expand Down
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ subprojects {
targetCompatibility = 1.8

//Latest version for generate
version = "0.9.9"
version = "0.10.0"

repositories {
mavenCentral()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@
*/
package com.ericsson.eiffel.remrem.generate.config;

import org.apache.commons.codec.binary.Base64;
import org.apache.commons.codec.binary.StringUtils;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.beans.factory.annotation.Value;
Expand Down Expand Up @@ -60,7 +58,7 @@ public class SecurityConfig extends WebSecurityConfigurerAdapter {
protected void configure(AuthenticationManagerBuilder auth) throws Exception {
LOGGER.debug("LDAP server url: "+ldapUrl);
auth.ldapAuthentication().userSearchFilter(userSearchFilter).contextSource().managerDn(managerDn).root(rootDn)
.managerPassword(decode(managerPassword)).url(ldapUrl);
.managerPassword(managerPassword).url(ldapUrl);
}

@Override
Expand All @@ -71,9 +69,4 @@ protected void configure(HttpSecurity http) throws Exception {

}

// To decrypt Base64 encode ldap manager password
private String decode(String password) {
return StringUtils.newStringUtf8(Base64.decodeBase64(password));
}

}

0 comments on commit 07e649c

Please sign in to comment.