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.

Sign up for GitHub

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

Jump to bottom

removed default actions and added evaluations semantics #193

Merged
merged 1 commit into from
Jan 7, 2025
Merged

removed default actions and added evaluations semantics #193

merged 1 commit into from
Jan 7, 2025

Conversation

ogazitt
Copy link
Collaborator

@ogazitt ogazitt commented Jan 2, 2025

This PR contains two substantive changes to the AuthZEN 1.0 spec:

  • Removes the default actions ("can_read", "can_write", etc), as a result of many pieces of feedback that this is confusing and doesn't add anything to the spec.
  • Adds the options.evaluations_semantic section to specify the execution semantics of the evaluations call, as agreed in the December 17 2024 AuthZEN call.

@netlify Netlify
Copy link

netlify bot commented Jan 2, 2025 •
edited
Loading

✅ Deploy Preview for authzen-todo canceled.

Name Link
🔨 Latest commit 28db3f7
🔍 Latest deploy log https://app.netlify.com/sites/authzen-todo/deploys/6776f385dd9f3b0007162f94

@davidjbrossard davidjbrossard merged commit b41f186 into main Jan 7, 2025
6 checks passed
@davidjbrossard davidjbrossard deleted the api/evaluations_semantics branch January 7, 2025 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidjbrossard davidjbrossard davidjbrossard approved these changes

@ggebel ggebel Awaiting requested review from ggebel

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ogazitt @davidjbrossard