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

Support for fine granted authz policies #650

Merged
merged 9 commits into from
Feb 28, 2022
Merged

Support for fine granted authz policies #650

merged 9 commits into from
Feb 28, 2022

Conversation

jkroepke
Copy link
Contributor

@jkroepke jkroepke commented Feb 28, 2022

What this PR does / why we need it:

Allow configure authz policies

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #648

Special notes for your reviewer:

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

@codecov
Copy link

codecov bot commented Feb 28, 2022

Codecov Report

Merging #650 (50ea524) into main (a33f5d7) will decrease coverage by 0.82%.
The diff coverage is 90.76%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #650      +/-   ##
============================================
- Coverage     96.09%   95.26%   -0.83%     
- Complexity     1267     1298      +31     
============================================
  Files            74       75       +1     
  Lines          4043     4116      +73     
  Branches        450      460      +10     
============================================
+ Hits           3885     3921      +36     
- Misses           73      103      +30     
- Partials         85       92       +7     
Impacted Files Coverage Δ
...s/keycloak/config/service/ClientImportService.java 94.37% <77.77%> (-1.04%) ⬇️
...nfig/service/ClientAuthorizationImportService.java 90.90% <90.90%> (ø)
...ak/config/exception/ImportProcessingException.java 100.00% <100.00%> (ø)
...s/keycloak/config/repository/ClientRepository.java 96.74% <100.00%> (+0.13%) ⬆️
...ys/keycloak/config/service/RealmImportService.java 100.00% <100.00%> (ø)
...onfig/repository/AuthenticationFlowRepository.java 81.96% <0.00%> (-11.48%) ⬇️
...eycloak/config/repository/ComponentRepository.java 59.09% <0.00%> (-11.37%) ⬇️
...ak/config/service/ExecutionFlowsImportService.java 87.78% <0.00%> (-4.59%) ⬇️
...ctory/UsedAuthenticationFlowWorkaroundFactory.java 94.17% <0.00%> (-2.12%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a33f5d7...50ea524. Read the comment docs.

@jkroepke jkroepke marked this pull request as ready for review February 28, 2022 23:13
@jkroepke jkroepke enabled auto-merge (squash) February 28, 2022 23:18
@jkroepke jkroepke merged commit e3a6404 into main Feb 28, 2022
@jkroepke jkroepke deleted the authz-policies branch February 28, 2022 23: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.

How to apply a policy from realm-management in client using the cli
1 participant