Skip to content

Feature/ab#2217 configurable roles#51

Merged
ztarbug merged 3 commits into
mainfrom
feature/ab#2217-configurable-roles
Jun 4, 2026
Merged

Feature/ab#2217 configurable roles#51
ztarbug merged 3 commits into
mainfrom
feature/ab#2217-configurable-roles

Conversation

@ztarbug

@ztarbug ztarbug commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

App now supports arbitrary external role names, that are mapped to standard internal role names.

Description

Application needs users to have one of three roles admin, user, reader. External names can now mapped via application.properties.

Motivation and Context

Standard role names are very generic and hard to use in multi application scenarios. Thus this PR presents a mapping mechanism.

How has this been tested?

Local Keycloak with sample users.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@ztarbug ztarbug requested a review from witchpou June 3, 2026 15:09

@witchpou witchpou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please examine this warning: Mockito is currently self-attaching to enable the inline-mock-maker. This will no longer work in future releases of the JDK. Please add Mockito as an agent to your build as described in Mockito's documentation: https://javadoc.io/doc/org.mockito/mockito-core/latest/org.mockito/org/mockito/Mockito.html#0.3

@ztarbug ztarbug merged commit c24839b into main Jun 4, 2026
2 checks passed
@ztarbug ztarbug deleted the feature/ab#2217-configurable-roles branch June 4, 2026 08:51
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.

2 participants