Skip to content

feat(global)!: opa validation#143

Merged
CptSchnitz merged 4 commits into
masterfrom
opa-validations
Jul 22, 2026
Merged

feat(global)!: opa validation#143
CptSchnitz merged 4 commits into
masterfrom
opa-validations

Conversation

@CptSchnitz

Copy link
Copy Markdown
Contributor

No description provided.

@CptSchnitz
CptSchnitz requested a review from ronenkapelian June 15, 2026 10:52
Comment thread policy/policy.rego
headers := lower_object_keys(input.headers)

tokens := [token |
# keys := data.possibleLocations[_]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we remove this comment?

Comment thread policy/policy.rego
]

claims := {"payload": payload, "valid": valid, "kid": kid} if {
# token := tokens[_]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same question

Comment thread policy/Makefile Outdated
test:
./opa test . -v

init:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove trailing whitespace after ":."

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

Comment thread policy/Makefile Outdated
@CptSchnitz
CptSchnitz merged commit 16b1dd5 into master Jul 22, 2026
7 of 8 checks passed
@CptSchnitz
CptSchnitz deleted the opa-validations branch July 22, 2026 07:26
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.

2 participants