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
GraphQL mutiny CompositeException instead of custom ErrorCode #15271
Comments
@phillip-kruger I think problem here in |
Hi @Andro999b Thanks, I'll look at this a.s.a.p |
Do you perhaps have a small reproducer? This might already be fix. If you can test against master, give that a go |
@phillip-kruger I have checked latest code in smallrey-graphql. And dont think that it fixed. Basically bug very simple to reproduce(also can send archive with repoducer). Just look at sample code that i left in description. |
Always easier for us if you can prepare a small archive reproducing the issue. We have a lot on our plate :). Thanks! |
Here is reproducer https://drive.google.com/file/d/1VEHmO6_spF8-QhPsBTUvHIJESt3KiAkF/view?usp=sharing |
Hi @Andro999b - Thanks for the reproducer. If you want to fix it please go ahead !! SmallRye GraphQL welcomes new contributors :) |
@phillip-kruger fix for this: smallrye/smallrye-graphql#653 |
Awesome ! Thanks @Andro999b |
Please close ticket when it will be available in quarkus. |
Late to the party, but the fix is good :-) (Mutiny does the right thing when it has to report a composite exception due to multiple failures) |
Any chance that fix will be included into quarkus 1.13 release? |
Yes |
Describe the bug
When using mutiny api with smallrye-graphql its not allow you to use custom ErrorCode
Expected behavior
return custom error code in response
Actual behavior
always return
io.smallrye.mutiny.CompositeException
Reproducer
https://drive.google.com/file/d/1VEHmO6_spF8-QhPsBTUvHIJESt3KiAkF/view?usp=sharing
Sample code
console output:
Environment (please complete the following information):
uname -a
orver
: Linux andrii-PC 5.4.0-65-generic Gizmo fixes and enhancements #73-Ubuntu SMP Mon Jan 18 17:25:17 UTC 2021 x86_64 x86_64 x86_64 GNU/Linuxjava -version
: openjdk version "11.0.9" 2020-10-20mvnw --version
orgradlew --version
): maven 3.6.3 or gradle 6.5.1The text was updated successfully, but these errors were encountered: