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

Added Keycloak 23.0.4, using getGroupByPath again and inverted briefRepresentation #977

Closed
wants to merge 3 commits into from

Conversation

jonasvoelcker
Copy link
Collaborator

@jonasvoelcker jonasvoelcker commented Jan 8, 2024

What this PR does / why we need it:

As Keycloak 23.0.4 inverted the briefRepresentation flag, this needs to fix it.

Which issue this PR fixes: fixes #976

Special notes for your reviewer:

  • Updated CI to use Keycloak 23.0.4
  • Changed briefRepresentation from false to true (mistakenly considered full: #25096)
  • Using getGroupByPath again after being fixed (#25111)

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

Copy link

codecov bot commented Jan 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.69%. Comparing base (efc3bfa) to head (cf512be).

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #977      +/-   ##
============================================
- Coverage     95.76%   95.69%   -0.07%     
+ Complexity     1366     1356      -10     
============================================
  Files            80       79       -1     
  Lines          4367     4346      -21     
  Branches        491      490       -1     
============================================
- Hits           4182     4159      -23     
- Misses           92       93       +1     
- Partials         93       94       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jonasvoelcker jonasvoelcker force-pushed the keycloak-23.0.4 branch 2 times, most recently from 7e56223 to 9d618c1 Compare January 8, 2024 22:43
Copy link

sonarqubecloud bot commented Jan 8, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@jonasvoelcker jonasvoelcker changed the title Added Keycloak 23.0.4 and inverted briefRepresentation Added Keycloak 23.0.4, using getGroupByPath again and inverted briefRepresentation Jan 8, 2024
@thomasdarimont
Copy link
Contributor

@jonasvoelcker thank you for the quick turnaround!

@jonasvoelcker
Copy link
Collaborator Author

Hi @st3v0rr, this one should be working, could you please try to rerun the failing build?

@jonasvoelcker jonasvoelcker mentioned this pull request Mar 8, 2024
1 task
Copy link

sonarqubecloud bot commented Mar 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@alwibrm
Copy link

alwibrm commented Mar 8, 2024

Hi @st3v0rr, any chance to get this merged soon?

@jonasvoelcker jonasvoelcker deleted the keycloak-23.0.4 branch March 12, 2024 06:23
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.

Keycloak 23.0.4 inverted briefRepresentation
3 participants