Skip to content

Commit

Permalink
removed hard coded reference to ou=applaccounts, need to correct unit…
Browse files Browse the repository at this point in the history
… tests
  • Loading branch information
Zeke Dean authored and Zeke Dean committed Oct 21, 2020
1 parent 6448cff commit 0e59390
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
6 changes: 6 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,12 @@ A tutorial with configuration examples is available in the [User Guide](docs/ind
./gradlew clean build shadowJar
```

#### Build without unit tests

```
./gradlew clean build shadowJar -x test
```

The result is `build/libs/kafka-ldap-integration-<version>.jar`, which contains the authentication and authorization classes, along with all of their dependencies.

**N.B.** This jar must be added to the classpath for the Kafka broker. The easist way to do that is to copy the jar into the directory `$KAFKA_HOME/libs`.
Expand Down
2 changes: 1 addition & 1 deletion src/main/kotlin/com/instaclustr/kafka/ldap/LDAPConfig.kt
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,6 @@ fun LDAPConfig.Config.toUserDN(user: String) = "$usrUid=$user,$usrBaseDN".toLowe
fun LDAPConfig.Config.toUserDNNodes(user: String) =
// assuming most use of Basta generated service accounts
listOf(
"$usrUid=$user,ou=ApplAccounts,$usrBaseDN".toLowerCase(),
"$usrUid=$user,$usrBaseDN".toLowerCase(),
"$usrUid=$user,$usrBaseDN".toLowerCase()
)

0 comments on commit 0e59390

Please sign in to comment.