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

OPA: Enable support for print() from Reno #3304

Open
mjungsbluth opened this issue Nov 8, 2024 · 0 comments
Open

OPA: Enable support for print() from Reno #3304

mjungsbluth opened this issue Nov 8, 2024 · 0 comments

Comments

@mjungsbluth
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
OPA allows to print() information from within the policy. This can be helpful to for example output conditions that led to a token being invalid and can cut down debugging times considerably.

Describe the solution you would like
Since we already have support for sending open tracing Spans for the OPA filters, it would be helpful to include the print() output that happened as part of an evaluation in the Span's events via the LogKV method.

Describe alternatives you've considered (optional)
The envoy plugin logs the print output which looses context to the decision made.

Additional context (optional)
This requires an upgrade to OPA v0.70 as the corresponding envoy plugin was changed to allow configuring a print hook. There is a test that demonstrates using such a hook.

Would you like to work on it?
Yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant