default-to-user-email #30

Open

stb wants to merge 7 commits from default-to-user-email into main

pull from: default-to-user-email

merge into: CCCHH:main

CCCHH:main

CCCHH:add-map-endpoint

CCCHH:refactor

CCCHH:testing

CCCHH:renovate/keycloak-packages

CCCHH:fix-role-membership

Conversation 2 Commits 7 Files changed 12 +665 -237

stb commented 2026-07-18 20:00:40 +02:00

Owner

Copy link

Flag default-to-user-email

When set for an export config, if the user doesn't have that attribute, their email will be substituted as a default. If any of the values is not a valid email address, they are filtered out.

Closes #29

Requires changes from #26.

Flag default-to-user-email When set for an export config, if the user doesn't have that attribute, their email will be substituted as a default. If any of the values is not a valid email address, they are filtered out. Closes #29 Requires changes from #26.

stb added 7 commits 2026-07-18 20:00:40 +02:00

Fix matchRole lookup to include group-inherited and composite roles ... 2c3c0a3dab

getRoleMembersStream only returns users with the role assigned
directly, so exported attributes silently excluded users who have
matchRole via group membership. Filter all realm users with
hasRole() instead, matching the resolution already used for authRole.

Add returning attributes split by name ... 464c02dfb0

Adds an endpoint that returns the list of attribute values as a map of attribute names in the requested attribute group.

This is useful for requesting the mailing list addresses, where the individual attributes apply to different mailing lists.

Merge branch 'main' into add-map-endpoint 9073ef086a

Finish merge a540e63bf3

Refactor and cleanup ... cd31ec8a20

* simplify AttributeEndpointsResourceProvider
* improve explanation of properties in AdminUiPage
* add example for mailing lists and map endpoint
* better explain configuration and operation of provider in main readme
* adjust the Keycloak realm
* Add unit test for endpoint class

Add running tests 1c2adb1a92

Flag default-to-user-email ... 149d98f90e

When set for an export config, if the user doesn't have that attribute, their email will be substituted as a default. If any of the values is not a valid email address, they are filtered out.

Closes #29

stb requested reviews from june, kritzl, lilly 2026-07-18 20:01:04 +02:00

lilly commented 2026-07-18 20:54:45 +02:00

Owner

Copy link

I unfortunately do not own any code of this repository, do not know any keycloak internals, and can add nothing to the review :|

I unfortunately do not own any code of this repository, do not know any keycloak internals, and can add nothing to the review :|

lilly refused to review 2026-07-18 20:54:49 +02:00

kritzl reviewed 2026-07-18 21:32:37 +02:00

All checks were successful

Hide all checks

/ Test (push) Successful in 56s

Details

/ Test (pull_request) Successful in 48s

Details

This pull request has changes conflicting with the target branch.

• .forgejo/workflows/test.yaml

View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Merge commit title Merge pull request 'default-to-user-email' (#30) from default-to-user-email into main

Merge commit body Reviewed-on: https://git.hamburg.ccc.de/CCCHH/keycloak-attribute-endpoints-provider/pulls/30

Merge pull request 'default-to-user-email' (#30) from default-to-user-email into main Reviewed-on: https://git.hamburg.ccc.de/CCCHH/keycloak-attribute-endpoints-provider/pulls/30 Copy merge commit message

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin default-to-user-email:default-to-user-email

git switch default-to-user-email

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff default-to-user-email

git switch default-to-user-email

git rebase main

git switch main

git merge --ff-only default-to-user-email

git switch default-to-user-email

git rebase main

git switch main

git merge --no-ff default-to-user-email

git switch main

git merge --squash default-to-user-email

git switch main

git merge --ff-only default-to-user-email

git switch main

git merge default-to-user-email

git push origin main

Sign in to join this conversation.

Reviewers

No reviewers

june

kritzl

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

3 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

CCCHH/keycloak-attribute-endpoints-provider!30

No description provided.