Sam Scott

Results 77 comments of Sam Scott

Hey all. Sorry for the slow response on this. Just added this capability for yall in 0.10.1: https://docs.rs/serde_qs/latest/serde_qs/struct.Serializer.html.

Hey @jneo8! That's a great idea, we can definitely consider adding that. In the meantime, it's actually not too bad to implement that yourself. Here's how `authorized_actions` works: https://github.com/osohq/oso/blob/0c5d33f7d9820cd6ae4842970edfd5735ae6ad4e/languages/python/oso/oso/oso.py#L152 So...

Hey @jneo8. Glad you have a workaround, but to make sure it's correct would you mind sharing the policy that caused this?

Thanks for sharing @jneo8! That generally looks good to me. One thing is that you might want to change the `has_role` check operate over the `org` not the `org.id`. E.g....

Offending code: https://github.com/osohq/oso/blob/main/polar-core/src/vm.rs#L2484-L2486 ``` // If the left rule has a specializer and the right does not, the left IS more specific, // so we return (Some(_), None) => return...

Thanks for opening this! I think the fix is either: - Stop auto-registering classes. This would make registering the constant first a hard failure. (Would appreciate feedback on whether this...

That's all spot on, thanks for the feedback. I think we probably can get rid of the auto-registration, but still support unregistered types.

Thanks for opening an issue! If there's an enough interest, we would love to do Clojure/Clojurescript libraries at some point. In the meantime, however, if you are interested in using...

Thanks! I'll look into that today, and setup cargo audit in ci to catch future issues.

Hey @ximon18. Thanks for reporting this! We've added cargo-audit to our CI in #647 and bumped some dependencies to address one of the issues. Some of the others are transitive...