Skip to content

KAFKA-5050 add ldap authentication support on PlainSaslServer#2833

Closed
nihed wants to merge 2 commits into
apache:trunkfrom
nihed:trunk
Closed

KAFKA-5050 add ldap authentication support on PlainSaslServer#2833
nihed wants to merge 2 commits into
apache:trunkfrom
nihed:trunk

Conversation

@nihed

@nihed nihed commented Apr 10, 2017

Copy link
Copy Markdown
Contributor

This feature will provide a way to add ldap authentication on PlainSaslServer.
Our Jaas Server configuration will be :
KafkaServer {
org.apache.kafka.common.security.plain.PlainLdapLoginModule REQUIRED
username="cent"
password="redhat"
source="ldap"
ldap_user_template="uid={0},ou=People,dc=srv,dc=world"
ldap_url="ldap://192.168.15.247:389";
};

I tested my code with existing tests and everything looks good. I will create another ticket later to add new unit tests.

@asfbot

asfbot commented Apr 10, 2017

Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/2853/
Test PASSed (JDK 8 and Scala 2.11).

@asfbot

asfbot commented Apr 10, 2017

Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.10/2848/
Test PASSed (JDK 7 and Scala 2.10).

@asfbot

asfbot commented Apr 10, 2017

Copy link
Copy Markdown

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/2849/
Test PASSed (JDK 8 and Scala 2.12).

@rajinisivaram

Copy link
Copy Markdown
Contributor

I think it will be better to do this on top of KIP-86 (PR #2022). A new callback handler can be defined for LDAP without changing PlainSaslServer. It will be easy to configure LDAP by specifying a different callback handler.

@wdm01github

Copy link
Copy Markdown

Can I get the compiled files? thanks

dongmingw@139.com

@asfgit

asfgit commented Nov 10, 2017

Copy link
Copy Markdown

FAILURE
No test results found.
--none--

@asfgit

asfgit commented Nov 10, 2017

Copy link
Copy Markdown

SUCCESS
4712 tests run, 0 skipped, 0 failed.
--none--

@asfgit

asfgit commented Nov 10, 2017

Copy link
Copy Markdown

FAILURE
4020 tests run, 1 skipped, 2 failed.
--none--

2 similar comments
@asfgit

asfgit commented Nov 10, 2017

Copy link
Copy Markdown

FAILURE
4020 tests run, 1 skipped, 2 failed.
--none--

@asfgit

asfgit commented Nov 10, 2017

Copy link
Copy Markdown

FAILURE
4020 tests run, 1 skipped, 2 failed.
--none--

@github-actions

Copy link
Copy Markdown

This PR is being marked as stale since it has not had any activity in 90 days. If you
would like to keep this PR alive, please leave a comment asking for a review. If the PR has
merge conflicts, update it with the latest from the base branch.

If you are having difficulty finding a reviewer, please reach out on the [mailing list](https://kafka.apache.org/contact).

If this PR is no longer valid or desired, please feel free to close it. If no activity occurs in the next 30 days, it will be automatically closed.

@github-actions github-actions Bot added the stale Stale PRs label Sep 13, 2024
@github-actions

Copy link
Copy Markdown

This PR has been closed since it has not had any activity in 120 days. If you feel like this
was a mistake, or you would like to continue working on it, please feel free to re-open the
PR and ask for a review.

@github-actions github-actions Bot added the closed-stale PRs that were closed due to inactivity label Oct 13, 2024
@github-actions github-actions Bot closed this Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

closed-stale PRs that were closed due to inactivity stale Stale PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants