ACL: Not specifying principalType crashes topics page

[RotemCDos]

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)

Creating a new ACL not in the following format principalType:principalName will cause the topics page to crash.
The entire page seems empty while the topics still exist.
For example: a new ACL test will be accepted by the UI but cause errors on the Kafka cluster.

Expected behavior

By the Apache Kafka ACL security validations (specified in clients/src/main/java/org/apache/kafka/common/utils/SecurityUtils.java), an ACL will not be created if it does not follow the principalType:principalName format.

Your installation details

I used the docker-compose with SSL/TLS, the latest Kafbat UI and Kafka versions for my current tests.
The issue exists on my production environment as well, which run on Kafbat UI v1.1.4 and Kafka 3.5.1.

Steps to reproduce

Navigate to the ACLs page -> Create ACL -> under Principal fill out test or any ACL without principalType: -> navigate to the topics page and it should now be empty

Screenshots

No response

Logs

No response

Additional context

No response

[github-actions]

Hi RotemCDos! 👋

Welcome, and thank you for opening your first issue in the repo!

Please wait for triaging by our maintainers.

As development is carried out in our spare time, you can support us by sponsoring our activities or even funding the development of specific issues.
Sponsorship link

If you plan to raise a PR for this issue, please take a look at our contributing guide.

[RotemCDos]

Additional information:

I have already created a fix for this issue on my local test fork and would love to create the PR next.
Attaching here a snippet of the validation function that is attached to the AclService.java file

  private void validatePrincipal(String principal) {
    if (principal == null || principal.isEmpty()) {
      throw new IllegalArgumentException("expected a string in format principalType:principalName but got " + principal);
    }

    String[] split = principal.split(":", 2);

    if (split.length != 2) {
      throw new IllegalArgumentException("expected a string in format principalType:principalName but got " + principal);
    }
  }

This is the exact same function that appears on Apache Kafka.