-
Notifications
You must be signed in to change notification settings - Fork 133
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
[WORKFLOW SDK BUG] Get workflow state raising exception for instances that do not exist #745
Comments
This issue has been automatically marked as stale because it has not had activity in the last 60 days. It will be closed in the next 7 days unless it is tagged (pinned, good first issue, help wanted or triaged/resolved) or other activity occurs. Thank you for your contributions. |
Not stale |
@ajstewart are you running this locally? and if so, can you share the dapr logs? |
There aren't any Dapr errors really, I'm assuming the SDK doesn't handle the not found. Here's the end of the trace that concerns the Dapr bit:
This is from just performing |
Hmm there should always be Dapr logs so maybe if you're running locally the Dapr logs are outputting to a file instead of the terminal? Is there a |
I guess I meant to say there are Dapr logs but nothing really comes up around this problem, this is the full output with debug level logs:
|
Hey @ajstewart , this error in fact comes from Dapr. It is a confusing error message but it has since been updated to |
Thanks @elena-kolevska. I guess it was because the docs said it should return |
Hey @ajstewart, here's a PR that addresses this: #771 |
The PR is now merged and will be a part of the next release (1.15.0). |
Expected Behavior
When I run:
I expect to receive a
None
response when the instance does not exist, as defined by the docs:Actual Behavior
I am getting an Exception when the instance does not exist:
Steps to Reproduce the Problem
The following small app can reproduce by calling the endpoint
Release Note
RELEASE NOTE:
The text was updated successfully, but these errors were encountered: