Do not decode query params in request filter

[janscheidegger]

Fixes wrongful decoding of query parameters when using quarkus-stork

See: https://quarkusio.zulipchat.com/#narrow/channel/294206-smallrye/topic/SmallRye.20Stork.20decodes.20URL.20Parameters

@aureamunoz

[geoand]

Thanks a lot for the contribution!

It would be great if there were a test included

[aureamunoz]

Thank you very much.
Yes! We would need a test for reproducing the bug before the fix and see the test green :-)

[geoand]

@janscheidegger sorry for not being more specific: We need a QuarkusUnitTest in https://github.com/quarkusio/quarkus/blob/main/extensions/resteasy-reactive/rest-client/deployment/src/test/java/io/quarkus/rest/client/reactive/stork.

[github-actions]

😭 Deploy PR Preview failed.

[aureamunoz]

cc @geoand @cescoffier

[quarkus-bot]

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit f129fad.

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

[geoand]

Thanks!

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 90c18c4.

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

📦 independent-projects/resteasy-reactive/server/vertx

❌ org.jboss.resteasy.reactive.server.vertx.test.sse.SseServerTestCase.shouldNotTryToSendToClosedSink - History

• 1 expectation failed. Response body doesn't match expectation. Expected: "true" Actual: false - java.lang.AssertionError

java.lang.AssertionError: 
1 expectation failed.
Response body doesn't match expectation.
Expected: "true"
  Actual: false

	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)

---

⚙️ Gradle Tests - JDK 17

📦 integration-tests/gradle

❌ io.quarkus.gradle.BuildConfigurationTest.buildNoOverride - History

• Multiple Failures (1 failure) -- failure 1 -- [sub project 'without-configuration', package type 'fast-jar'] Expecting path: - org.assertj.core.error.AssertJMultipleFailuresError

org.assertj.core.error.AssertJMultipleFailuresError: 

Multiple Failures (1 failure)
-- failure 1 --
[sub project 'without-configuration', package type 'fast-jar'] 
Expecting path:
  /home/runner/_work/quarkus/quarkus/integration-tests/gradle/target/classes/build-configuration/without-configuration/build/quarkus-app/quarkus-run.jar
to exist (symbolic links were followed).