Correctly handle REST field injection

[geoand]

This is done by converting the Resource class to
@RequestScoped when possible and failing the build when it's not

• Fixes: [resteasy-reactive] [3.8.6] Header not reset between request #45789

[FroMage]

I'm not sure why the inherited fields don't make the class auto-@RequestScoped.

[geoand]

I'm not sure why the inherited fields don't make the class auto-@RequestScoped.

The point of the test is to show that the auto-conversion mechanism is best effort. In this specific case we could do extra work to figure out that field injection is needed, but it will never be as complete as the EndpointIndexer and thus we need to be able to fail the build when the best effort conversion was not enough

[geoand]

This is a breaking change, but I think it's backport is warranted @gsmet @cescoffier

[FroMage]

The point of the test is to show that the auto-conversion mechanism is best effort. In this specific case we could do extra work to figure out that field injection is needed, but it will never be as complete as the EndpointIndexer and thus we need to be able to fail the build when the best effort conversion was not enough

OK, we can always improve later

[geoand]

I actually might have to do it now because some of the TCK tests are failing...

[geoand]

PR updated. We should actually fail now in far fewer cases

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 9eb48b4.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.