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

Allow configurable authorization role-mappers #1812

Open
ryanemerson opened this issue Apr 24, 2023 · 0 comments
Open

Allow configurable authorization role-mappers #1812

ryanemerson opened this issue Apr 24, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@ryanemerson
Copy link
Contributor

ryanemerson commented Apr 24, 2023

The operator automatically configures the role-mapper AND the credentials used by the controllers based upon the client cert strategy configured in the Infinispan CR. The follow role-mapper configurations are applied:

clientCert: None | Validate -> cluster-role-mapper
clientCert: Authenticate -> common-name-role-mapper

A valid use-case is for a user to adopt the clientCert: validate strategy with a common-name-role-mapper to ensure that the CN of the certificate determines the capabilities of the client(s).

Currently it's not possible for the user to define a custom role-mapper via custom server configuration, as the Operator controllers need to be aware of the desired role-mapper in order to configure their rest client correctly.

We should add an optional field to the Infinispan CR that allows the role-mapper to be explicitly configured, with the previous defaults applied if the field is omitted.

Example Infinispan CR configuration:

spec:
  security:
    authorization:
      roleMapper: common-name-role-mapper
@ryanemerson ryanemerson added the bug Something isn't working label Apr 24, 2023
@ryanemerson ryanemerson changed the title Client Cert Validate causes 403 ISPN000287: Unauthorized access Allow configurable authorization role-mappers Apr 24, 2023
@ryanemerson ryanemerson added enhancement New feature or request and removed bug Something isn't working labels Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant