-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
2.14.2 backports 3 #29474
Merged
Merged
2.14.2 backports 3 #29474
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
(cherry picked from commit 97b10e5)
- resolves quarkusio#29230 (cherry picked from commit 61b9671)
… have been added programmatically (cherry picked from commit a11e121)
(cherry picked from commit 68a5252)
… guide Since 2.12.1.Final, a @restclient qualifier is needed, see quarkusio#28004 Signed-off-by: Harald Albers <github@albersweb.de> (cherry picked from commit 78cb98c)
…n methods Although we can make this work in the future, it involves a fair amount of work for a very small gain, so let's make it explicit for now that this combination is not allowed. Follows up on: quarkusio#29118 (cherry picked from commit 930744b)
…larative filter annotations (cherry picked from commit 3a5ad2d)
…ion by default (cherry picked from commit d9bb2d0)
…ors when there are other interceptors in play (cherry picked from commit d5ec1f2)
The `EventBusCodecProcessor` used to create a map from Jandex `Type` of message to a `DotName` of the corresponding codec, and then expose that map as a repeatable `MessageCodecBuildItem`. When creating each build item, the code used to call `Type.toString()`, which is never the right thing to do when obtaining a class name from Jandex `Type`. With this commit, the map is from type `DotName` to codec `DotName`, and the key in the map is obtained by `Type.name()`. (cherry picked from commit bba49af)
(cherry picked from commit 537e81b)
Sometimes, ArC used to call the `AnnotationLiteralProcessor` with annotations that are either class-retained, or their classes are missing (which is perfectly legal). It is a minority of callers though -- most of them only call `AnnotationLiteralProcessor` with qualifiers or interceptor bindings. Those must always be runtime-retained and their classes must be present, and it is an error if they are not. At the same time, we don't want to generate annotation literals for class-retained annotations or for annotations whose classes are missing. The `AnnotationLiteralProcessor` already checks that the annotation class exists, so with this commit, it also checks that the annotation for which the literal is generated is runtime-retained. If not, an exception is thrown. This lets us know that there's a problem somewhere. There are 2 places where ArC deals with arbitrary annotations and may need to generate annotation literals for them; in both cases, the code has to process all annotations present on an injection point. In these cases, it is straightforward to filter out class-retained annotations and missing annotation classes. (cherry picked from commit 46268e3)
The combination of these two HTTP headers is illegal, so let's make sure RESTEasy Reactive does not send the Content-Length header when Transfer-Encoding is set Fixes: quarkusio#29059 (cherry picked from commit b632c9e)
Bumps [postgresql](https://github.com/pgjdbc/pgjdbc) from 42.5.0 to 42.5.1. - [Release notes](https://github.com/pgjdbc/pgjdbc/releases) - [Changelog](https://github.com/pgjdbc/pgjdbc/blob/master/CHANGELOG.md) - [Commits](pgjdbc/pgjdbc@REL42.5.0...REL42.5.1) --- updated-dependencies: - dependency-name: org.postgresql:postgresql dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> (cherry picked from commit 7bf72c6)
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/maven
area/resteasy-reactive
area/vertx
labels
Nov 24, 2022
…f subtyping Fixes quarkusio#29227 (cherry picked from commit 433a289)
(cherry picked from commit de1b8d4)
quarkus-bot
bot
added
the
area/platform
Issues related to definition and interaction with Quarkus Platform
label
Nov 24, 2022
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/infra
internal and infrastructure related issues
area/maven
area/platform
Issues related to definition and interaction with Quarkus Platform
area/resteasy-reactive
area/vertx
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please don't merge, I will merge it myself.