Skip to content

Commit

Permalink
Merge pull request #1703 from brianaydemir/issue-1701
Browse files Browse the repository at this point in the history
Fix testing for overlap bewteen OIDC vs. required groups
  • Loading branch information
bbockelm authored Nov 15, 2024
2 parents d119d27 + 1764853 commit ce4615b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion oa4mp/resources/policies.qdl
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ group_list. := claims.groups;
remove(claims.groups);

{{ if .GroupRequirements }}
if [0 == size(|^group_list. \/ { {{- range $idx, $grp := .GroupRequirements -}}{{- if eq $idx 0 -}}'{{- $grp -}}'{{else}}, '{{- $grp -}}'{{- end -}}{{- end -}} })] then
if [0 == size(|^group_list. /\ { {{- range $idx, $grp := .GroupRequirements -}}{{- if eq $idx 0 -}}'{{- $grp -}}'{{else}}, '{{- $grp -}}'{{- end -}}{{- end -}} })] then
[
sys_err.ok := false;
sys_err.message := 'Authenticated user is not in any of the following groups: {{ range $idx, $grp := .GroupRequirements -}}{{- if eq $idx 0 -}}"{{- $grp -}}"{{else}}, "{{- $grp -}}"{{- end -}}{{- end -}}';
Expand Down

0 comments on commit ce4615b

Please sign in to comment.