Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.2.8 backports 4 #36834

Merged
merged 20 commits into from
Nov 3, 2023
Merged

3.2.8 backports 4 #36834

merged 20 commits into from
Nov 3, 2023

Conversation

aloubyansky
Copy link
Member

@rsvoboda @gastaldi @gsmet this is the remaining big batch. There are 4 remaining PRs marked with backport-3.2.
And we may want to upgrade to Keycloak 22.0.6 (planned to be released next week) from 22.0.5 to match the version of the RHBK release.

viniciusfcf and others added 20 commits November 2, 2023 13:35
Fix Authorization of Web Endpoints link

(cherry picked from commit e4c9dc8)
…gate.parse(..) within resteasy-reactive

Previously, a "broken" MIME-type in an access header could trigger an StringIndexOutOfBoundsException during MediaTypeHeaderDelegate.parse(..) instead of the more suitable IllegalArgumentException.

Example: "Accept: x; /x"

This PR now throws an IllegalArgumentException in case of a broken MIME-type like in the example.

Fixes quarkusio#36159

(cherry picked from commit 84d6c5d)
(cherry picked from commit ac93aff)
Default response content type header should conform to [GraphQL over HTTP spec](https://github.com/graphql/graphql-over-http/blob/main/spec/GraphQLOverHTTP.md#serialization-format).

(cherry picked from commit bff1fb7)
As usual with Db2, we are shooting in the dark a bit but it seems to
solve the OP's issue.

Fixes quarkusio#36399

(cherry picked from commit 78d5bc3)
Apply suggestions from code review

Co-authored-by: Yoann Rodière <[email protected]>

Removing redundant contant

Applying jmfinelli's suggestions for a better wording

Signed-off-by: Michal Maléř <[email protected]>
(cherry picked from commit 6dabad6)
Without this change interceptors meant to be
used on the client were being used on the
server as well

(cherry picked from commit 8207192)
(cherry picked from commit 2f453f4)
@quarkus-bot quarkus-bot bot added area/arc Issue related to ARC (dependency injection) area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/graphql area/health area/liquibase area/maven area/oidc area/persistence OBSOLETE, DO NOT USE labels Nov 2, 2023
@quarkus-bot quarkus-bot bot added area/platform Issues related to definition and interaction with Quarkus Platform area/resteasy-classic area/rest area/smallrye area/testing labels Nov 2, 2023
Copy link

github-actions bot commented Nov 2, 2023

🙈 The PR is closed and the preview is expired.

This comment has been minimized.

Copy link

quarkus-bot bot commented Nov 3, 2023

✔️ 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.

@aloubyansky aloubyansky merged commit 1789356 into quarkusio:3.2 Nov 3, 2023
49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/arc Issue related to ARC (dependency injection) area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/graphql area/health area/liquibase area/maven area/oidc area/persistence OBSOLETE, DO NOT USE area/platform Issues related to definition and interaction with Quarkus Platform area/rest area/resteasy-classic area/smallrye area/testing
Projects
Development

Successfully merging this pull request may close these issues.