Smallrye GraphQL native: SPI ErrorExtensionProvider implementations not included

[schulzp]

Describe the bug

I use quarkus-smallrye-graphql on top op quarkus 2.11.2.Final. Since I upgraded from 2.9.2.Final my integration tests started to fail: I do have ITs that make the application throw an exception, for example, when validation fails, or writing to the database fails. Those tests now fail due to a timeout on the GQL client end. On the server side I see the following exception:

ServiceConfigurationError: failed to load io.smallrye.graphql.api.ErrorExtensionProvider, class io.smallrye.graphql.execution.error.ErrorCodeExtensionProvider does not exist

Expected behavior

Application should fail with expected (gracefully handled) exception.

Actual behavior

Application does not close response which leads to timeout on client side.

How to Reproduce?

1. Create Quarkus Application with SmallRye GraphQL extension
2. Annotate at least one method with @Mutation that throws an exception
3. Execute that mutation (via HTTP client or generated GQL client)

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Quarkus version or git rev

2.11.2.Final

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.8.1 (05c21c65bdfed0f71a2f2ada8b84da59348c4c5d)

Additional information

smallrye-graphql-1.6.0.jar comes with META-INF/services/io.smallrye.graphql.api.ErrorExtensionProvider, the classes referenced (io.smallrye.graphql.execution.error.ErrorCodeExtensionProvider and io.smallrye.graphql.execution.error.ExceptionNameErrorExtensionProvider) do not end up in the native binary, however.

If I manually add the following entries to my reflect-config.json, everything works as expected again:

{
  "name":"io.smallrye.graphql.execution.error.ErrorCodeExtensionProvider",
  "methods":[{"name":"<init>","parameterTypes":[] }]
},
{
  "name":"io.smallrye.graphql.execution.error.ExceptionNameErrorExtensionProvider",
  "methods":[{"name":"<init>","parameterTypes":[] }]
}

[quarkus-bot]

/cc @Ladicek, @jmartisk, @phillip-kruger, @radcortez

[phillip-kruger]

Thanks, I'll fix this