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

Add option for specifying decision ID to SDK #6100

Closed
brianchhun-chime opened this issue Jul 17, 2023 · 0 comments · Fixed by #6101
Closed

Add option for specifying decision ID to SDK #6100

brianchhun-chime opened this issue Jul 17, 2023 · 0 comments · Fixed by #6101

Comments

@brianchhun-chime
Copy link
Contributor

What is the underlying problem you're trying to solve?

It could be useful to allow clients to pass-through their own decision ID into an SDK decision call rather than having OPA generate one for them. At least for my use-case, clients of my service already provide us with a decision ID and our service would like to simply forward it along to OPA so that decision logs also get tagged with that same decision ID. Right now, we have code to ignore the OPA decision ID in favor of our client-generated ID, which is a bit messy to maintain.

Describe the ideal solution

I think if clients could specify the decision ID for the decisions returned by the Decision and Partial functions provided by the SDK package that would be ideal. And if a decision ID is not provided, a uniquely generated identifier would be generated for the decision, just as before.

Describe a "Good Enough" solution

I think the ideal solution doesn't seem too difficult to achieve and it's also good enough.

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

Successfully merging a pull request may close this issue.

1 participant