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

cedar-go allows arbitrary entity id in Action, but aws rust implementation only supports namespace 'Action' #11

Open
erichoffmanstrongdm opened this issue Apr 28, 2024 · 4 comments

Comments

@erichoffmanstrongdm
Copy link
Collaborator

No description provided.

@philhassey
Copy link
Collaborator

Can you go into a bit more detail on this?

How is Something::Action::"example" rejected by Rust?

Is Action::Something::"example" also rejected by Rust?

@erichoffmanstrongdm
Copy link
Collaborator Author

erichoffmanstrongdm commented Apr 29, 2024

oh, this is actually all over. there are validators that ensure for example that any member of []parents of an action entity are also actions.

but line 721 (action_constraint_contains_only_action_types) in cedar-policy-core/src/parser/cst_to_ast.rs is a pretty big gate

https://github.com/cedar-policy/cedar/blob/main/cedar-policy-core/src/parser/cst_to_ast.rs#L717

@philhassey
Copy link
Collaborator

Can you make that a link, thanks!

@erichoffmanstrongdm erichoffmanstrongdm changed the title ceadar-go allows arbitrary entity id in Action, but aws rust implementation only supports namespace 'Action' cedar-go allows arbitrary entity id in Action, but aws rust implementation only supports namespace 'Action' Apr 29, 2024
@erichoffmanstrongdm
Copy link
Collaborator Author

I have validated that cedar is perfectly happy to operate on any namespace prefix (including unknown), but requires that Action be the last element in the namespace. this kind of contradicts the notion that namespaces are just unstructured strings that accidently have colons in them

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

No branches or pull requests

2 participants