-
Notifications
You must be signed in to change notification settings - Fork 93
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
Logging error #2230
Comments
This will be fixed with jboss-logging/jboss-logmanager#493 |
@phillip-kruger I'm not entirely sure this will get fixed by the Log Manager update as I was using a log method with no formatting at all when having the issue and you're using |
@gsmet yes this issue was not fixed with jboss-logging/jboss-logmanager#493 @dmlloyd can you help ? I include I reproducer here: After starting the app (dev mode is fine) run the following query:
This should give you:
|
I have some time before calling it a day, let me try to diagnose this a bit more precisely. |
This was failing only in dev mode when using printf-style logging. Fixes smallrye/smallrye-graphql#2230
quarkusio/quarkus#44834 should fix it. |
^ has been merged. |
This was failing only in dev mode when using printf-style logging. Fixes smallrye/smallrye-graphql#2230 (cherry picked from commit 64f0108)
Thanks @gsmet ! |
This was failing only in dev mode when using printf-style logging. Fixes smallrye/smallrye-graphql#2230 (cherry picked from commit 64f0108)
With this code:
and this query:
I get the following error:
The text was updated successfully, but these errors were encountered: