-
Notifications
You must be signed in to change notification settings - Fork 92
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. |
With this code:
and this query:
I get the following error:
The text was updated successfully, but these errors were encountered: