Skip to content

Comments

fix: add non-standard default roles to filtering#492

Merged
lechnerc77 merged 2 commits intomainfrom
fix/default-roles
Oct 21, 2025
Merged

fix: add non-standard default roles to filtering#492
lechnerc77 merged 2 commits intomainfrom
fix/default-roles

Conversation

@lechnerc77
Copy link
Member

Purpose

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[X] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Test the code via automated test
make test

What to Check

Verify that the following are valid:

  • Automated tests are executed successfully

Other Information

Some default rolöes doe not follow the standard XSUAA pattern of having IsReadOnly true and empty attribute list. A case by case clarification is needed to identify such roles and exclude them from export.

Checklist for reviewer

The following organizational tasks must be completed before merging this PR:

  • The PR status on the Project board is set (typically "in review").
  • The PR has the matching labels assigned to it.
  • If the PR closes an issue, the issue is referenced.
  • Possible follow-up issues are created and linked.

@github-actions github-actions bot added this to the 1.5.0 milestone Oct 20, 2025
@github-actions github-actions bot added the bug Something isn't working label Oct 20, 2025
@lechnerc77 lechnerc77 enabled auto-merge (squash) October 20, 2025 14:05
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@lechnerc77 lechnerc77 merged commit 09a06c5 into main Oct 21, 2025
12 of 13 checks passed
@lechnerc77 lechnerc77 deleted the fix/default-roles branch October 21, 2025 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Error when re-creating subaccount due to read-only role

2 participants