libre.sh/keycloak-scim
2
0
Fork 0
Code Issues 14 Pull requests Releases Packages Activity Actions 6

Update RolePolicyRepresentation fields from 'boolean' to 'Boolean'

Browse source 
closes #32117

Signed-off-by: mposolda <mposolda@gmail.com>
This commit is contained in:
mposolda 2024-08-14 11:24:22 +02:00 committed by Marek Posolda
parent cb6eb187ac
commit 54a538b3ad
3 changed files with 14 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
authz/policy/common/src/main/java/org/keycloak/authorization/policy/provider/role/RolePolicyProvider.java
View file

@ -62,9 +62,10 @@ public class RolePolicyProvider implements PolicyProvider {
RoleModel role = realm.getRoleById(roleDefinition.getId());
if (role != null) {
boolean hasRole = hasRole(identity, role, realm, authorizationProvider, policyRep.isFetchRoles());
boolean isFetchRoles = policyRep.isFetchRoles() != null && policyRep.isFetchRoles();
boolean hasRole = hasRole(identity, role, realm, authorizationProvider, isFetchRoles);
if (!hasRole && roleDefinition.isRequired()) {
if (!hasRole && roleDefinition.isRequired() != null && roleDefinition.isRequired()) {
evaluation.deny();
return;
} else if (hasRole) {

2
authz/policy/common/src/main/java/org/keycloak/authorization/policy/provider/role/RolePolicyProviderFactory.java
View file

@ -142,7 +142,9 @@ public class RolePolicyProviderFactory implements PolicyProviderFactory<RolePoli
}
private void updateRoles(Policy policy, RolePolicyRepresentation representation, AuthorizationProvider authorization) {
if (representation.isFetchRoles() != null) {
policy.putConfig("fetchRoles", String.valueOf(representation.isFetchRoles()));
}
updateRoles(policy, authorization, representation.getRoles());
}

16
core/src/main/java/org/keycloak/representations/idm/authorization/RolePolicyRepresentation.java
View file

@ -25,7 +25,7 @@ import java.util.Set;
public class RolePolicyRepresentation extends AbstractPolicyRepresentation {
private Set<RoleDefinition> roles;
private boolean fetchRoles;
private Boolean fetchRoles;
@Override
public String getType() {
@ -40,7 +40,7 @@ public class RolePolicyRepresentation extends AbstractPolicyRepresentation {
this.roles = roles;
}
public void addRole(String name, boolean required) {
public void addRole(String name, Boolean required) {
if (roles == null) {
roles = new HashSet<>();
}
@ -59,24 +59,24 @@ public class RolePolicyRepresentation extends AbstractPolicyRepresentation {
addRole(clientId + "/" + name, required);
}
public boolean isFetchRoles() {
public Boolean isFetchRoles() {
return fetchRoles;
}
public void setFetchRoles(boolean fetchRoles) {
public void setFetchRoles(Boolean fetchRoles) {
this.fetchRoles = fetchRoles;
}
public static class RoleDefinition implements Comparable<RoleDefinition> {
private String id;
private boolean required;
private Boolean required;
public RoleDefinition() {
this(null, false);
}
public RoleDefinition(String id, boolean required) {
public RoleDefinition(String id, Boolean required) {
this.id = id;
this.required = required;
}
@ -89,11 +89,11 @@ public class RolePolicyRepresentation extends AbstractPolicyRepresentation {
this.id = id;
}
public boolean isRequired() {
public Boolean isRequired() {
return required;
}
public void setRequired(boolean required) {
public void setRequired(Boolean required) {
this.required = required;
}