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

Decision log path for query against v1/data not correct #1532

Closed
tsandall opened this issue Jun 28, 2019 · 1 comment
Closed

Decision log path for query against v1/data not correct #1532

tsandall opened this issue Jun 28, 2019 · 1 comment
Assignees
Labels

Comments

@tsandall
Copy link
Member

Expected Behavior

If clients are executing queries against the root of the data namespace then the decision log path should be empty. Note, this change will be backwards incompatible however it's unlikely that anyone is relying on this behavior today since clients typically query for specific decision paths OR they use the main decision (both of which are logged correctly.)

Actual Behavior

Currently the decision log path is set to data because of how the path prefix is stripped.

Steps to Reproduce the Problem

  1. Run opa with decision logging enabled: opa run -s --set decision_logs.console=true
  2. Execute a query against v1/data: curl localhost:8181/v1/data
  3. Observe path in decision log event
@tsandall tsandall added the bug label Jun 28, 2019
@patrick-east patrick-east self-assigned this Dec 17, 2019
@patrick-east
Copy link
Contributor

This gets fixed as part of #1958

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

No branches or pull requests

2 participants