Properly support Optional as a header param for REST Client

[geoand]

• Fixes: Optional<> @HeaderParam REST Client arguments #47366

[Postremus]

I believe this code is copied for sub resources in handleSubResourceMethod starting around line 1680. Could you please add a test for subresources?

[geoand]

Damn, I hate those rubresource :).

[geoand]

As expected, those subresources are causing all sorts of problems .... Should be fixed now

[geoand]

FWIW, I would hold off backporting the second commit to 3.20 for the time being

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 3ad6e97.

✅ 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.

---

Flaky tests - Develocity

⚙️ JVM Tests - JDK 17

📦 extensions/micrometer/deployment

✖ io.quarkus.micrometer.deployment.binder.VertxConnectionMetricsTest.testConnectionMetrics - History

• event executor terminated - java.util.concurrent.RejectedExecutionException

java.util.concurrent.RejectedExecutionException: event executor terminated
	at io.netty.util.concurrent.SingleThreadEventExecutor.reject(SingleThreadEventExecutor.java:934)
	at io.netty.util.concurrent.SingleThreadEventExecutor.offerTask(SingleThreadEventExecutor.java:353)
	at io.netty.util.concurrent.SingleThreadEventExecutor.addTask(SingleThreadEventExecutor.java:346)
	at io.netty.util.concurrent.SingleThreadEventExecutor.execute(SingleThreadEventExecutor.java:836)
	at io.netty.util.concurrent.SingleThreadEventExecutor.execute0(SingleThreadEventExecutor.java:827)
	at io.netty.util.concurrent.SingleThreadEventExecutor.execute(SingleThreadEventExecutor.java:817)
	at io.vertx.core.impl.EventLoopExecutor.execute(EventLoopExecutor.java:35)

[gsmet]

@Postremus could you have another look?

[Postremus]

Thank you for muddling through the sub resources handling!

We can add support for optional for other parameter types (e.g. Query) in a follow up PR.

[geoand]

Makes sense!