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

ACL: ACLs with leading/trailing spaces in principal names not shown in table #590

Closed
4 tasks done
polegkashti opened this issue Oct 8, 2024 · 0 comments · Fixed by #591
Closed
4 tasks done

ACL: ACLs with leading/trailing spaces in principal names not shown in table #590

polegkashti opened this issue Oct 8, 2024 · 0 comments · Fixed by #591
Assignees
Labels
area/acl scope/frontend Related to frontend changes status/triage/completed Automatic triage completed type/bug Something isn't working
Milestone

Comments

@polegkashti
Copy link
Contributor

Issue submitter TODO list

  • I've looked up my issue in FAQ
  • I've searched for an already existing issues here
  • I've tried running main-labeled docker image and the issue still persists there
  • I'm running a supported version of the application which is listed here

Describe the bug (actual behavior)

When an ACL is created with trailing or leading spaces (e.g., in the principal name), it does not appear in the ACL table on the Kafka UI. However, the ACL is correctly saved in Kafka with the spaces, and this discrepancy only becomes evident when using the Kafka CLI. This makes it difficult to identify ACLs with spaces through the UI.

Expected behavior

The ACL table in the Kafka UI should display the ACLs exactly as they are saved in Kafka, including any trailing or leading spaces in principal names or other fields. This ensures that users can see ACLs as they exist in Kafka, avoiding confusion.

Your installation details

.

Steps to reproduce

  1. Create a new ACL with trailing or leading spaces in the principal name (e.g., "user " or " user").
  2. Go to the ACL page in Kafka UI.
  3. Observe that the newly created ACL is not displayed in the table.
  4. Use the Kafka CLI or F12 to check the saved ACL and note that it exists with the spaces.

Screenshots

No response

Logs

No response

Additional context

No response

@polegkashti polegkashti added status/triage Issues pending maintainers triage type/bug Something isn't working labels Oct 8, 2024
@kapybro kapybro bot added status/triage/manual Manual triage in progress status/triage/completed Automatic triage completed and removed status/triage Issues pending maintainers triage labels Oct 8, 2024
polegkashti pushed a commit to polegkashti/kafka-ui that referenced this issue Oct 8, 2024
@Haarolean Haarolean linked a pull request Oct 8, 2024 that will close this issue
13 tasks
@Haarolean Haarolean added scope/frontend Related to frontend changes area/acl and removed status/triage/manual Manual triage in progress labels Oct 8, 2024
@Haarolean Haarolean changed the title ACLs with leading/trailing spaces in principal names not shown in table ACL: ACLs with leading/trailing spaces in principal names not shown in table Oct 8, 2024
@Haarolean Haarolean added this to the 1.1 milestone Oct 9, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in Release 1.1 Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/acl scope/frontend Related to frontend changes status/triage/completed Automatic triage completed type/bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants