You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
paulbakker
changed the title
documentation: error handling example is outdated for Java 21
documentation: error handling example is outdated for currentl graphql-java version
Mar 5, 2024
Out of curiosity for someone who found this issue as they were upgrading versions and has a reference to that exact line of code in their codebase that's causing strife, what is the recommended line of code here (until of course the docs are updated) for a default handling of exceptions? Should we just not have a default case?
I solved this by creating an instance of SimpleDataFetcherExceptionHandler myself, and then calling simpleDataFetcherExceptionHandler.handleException() instead of super.handleException(). Similar to how the initial implementation looked like.
What are you missing in the docs
The error-handling page shows an example which ends with a delegate call to handle other exceptions then
MyException
.`return DataFetcherExceptionHandler.super.handleException(handlerParameters);
This default method was however deprecated and is removed in
graphql-java
21.3, graphql-java/graphql-java@1021219The text was updated successfully, but these errors were encountered: