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

PATCH - invalid HTTP verb #612

Closed
davidfigueredo-ink opened this issue Jul 29, 2024 · 0 comments · Fixed by #632
Closed

PATCH - invalid HTTP verb #612

davidfigueredo-ink opened this issue Jul 29, 2024 · 0 comments · Fixed by #632

Comments

@davidfigueredo-ink
Copy link

davidfigueredo-ink commented Jul 29, 2024

Expected Behavior

I'm trying to call with a PATCH request to another microservice using the Dapr client invoke.

Actual Behavior

When I call the Dapr client invoker with the PATCH verb, it throws the following error
Error: 13 INTERNAL: failed to invoke, id: my_microservice, err: rpc error: code = Internal desc = error invoking app channel: rpc error: code = InvalidArgument desc = invalid HTTP verb

Steps to Reproduce the Problem

Call the GRPCClientInvoker class invoke with a PATCH request.

Possible Solution

On the enum of verbs for the HTTP requests you have the PATCH type, but, in the HttpVerb.util.js, the PATCH is not defined on the switch cases of the two methods. I think that adding the case for that verb will be a possible solution.

Thanks in advance!

johnvan7 added a commit to johnvan7/js-sdk that referenced this issue Sep 18, 2024
fix dapr#612

Signed-off-by: Giovanni Vella (johnvan7)<[email protected]>
@johnvan7 johnvan7 mentioned this issue Sep 18, 2024
3 tasks
yaron2 pushed a commit that referenced this issue Sep 18, 2024
fix #612

Signed-off-by: Giovanni Vella (johnvan7)<[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant