Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added login page #282

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

added login page #282

wants to merge 5 commits into from

Conversation

johnneal3
Copy link

I have a requirement on my end to secure access to Kafdrop via a login page. I am submitting this code for your review / approval. This is a simple login solution, only supporting a single user.

Default:

  • username = admin
  • password = admin

Override via command line args (validated in docker/docker-compose as well):

java --add-opens=java.base/sun.nio.ch=ALL-UNNAMED    \
     -jar target/kafdrop-3.28.0-SNAPSHOT.jar \
      --kafka.brokerConnect=localhost:29092    \
      --spring.security.user.name=fred --spring.security.user.password=mary

image

@lgtm-com
Copy link

lgtm-com bot commented May 24, 2021

This pull request introduces 1 alert when merging bed6777 into fdc887a - view on LGTM.com

new alerts:

  • 1 for Disabled Spring CSRF protection

johnneal3 and others added 4 commits May 24, 2021 16:33
corrected issue caused by enabling http.csrf() that was not caught in initial testing.
fixed issue with delete topic after http.csrf() was enabled with addition of login screen
@xtruan
Copy link

xtruan commented Dec 10, 2021

I have tested this enhancement (verifying login prompt, creating topics, deleting topics, etc.) at my organization and we have incorporated into our Kafdrop instance. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants