Jackson: also detect class referenced by @JsonTypeIdResolver
#27346
Labels
Milestone
Comments
|
@snazy yes, it makes perfect sense. |
snazy
added a commit
to snazy/quarkus
that referenced
this issue
Aug 18, 2022
Adds missing support for classes referenced by `@JsonTypeIdResolver`, which is expecially important when using native builds. Fixes quarkusio#27346
snazy
added a commit
to snazy/quarkus
that referenced
this issue
Aug 18, 2022
Adds missing support for classes referenced by `@JsonTypeIdResolver`, which is particularly important when using native builds. The added integration tests are likely only meaningful when run in native mode (`-Pnative`), otherwise "good old Java reflection" does its job. Fixes quarkusio#27346
snazy
added a commit
to snazy/quarkus
that referenced
this issue
Aug 18, 2022
Adds missing support for classes referenced by `@JsonTypeIdResolver`, which is particularly important when using native builds. The added integration tests are likely only meaningful when run in native mode (`-Pnative`), otherwise "good old Java reflection" does its job. Fixes quarkusio#27346
|
@gsmet created the PR |
sheilamjones
pushed a commit
to sheilamjones/quarkus
that referenced
this issue
Aug 22, 2022
Adds missing support for classes referenced by `@JsonTypeIdResolver`, which is particularly important when using native builds. The added integration tests are likely only meaningful when run in native mode (`-Pnative`), otherwise "good old Java reflection" does its job. Fixes quarkusio#27346
gsmet
pushed a commit
to gsmet/quarkus
that referenced
this issue
Aug 23, 2022
Adds missing support for classes referenced by `@JsonTypeIdResolver`, which is particularly important when using native builds. The added integration tests are likely only meaningful when run in native mode (`-Pnative`), otherwise "good old Java reflection" does its job. Fixes quarkusio#27346 (cherry picked from commit e4e0d4e)
fercomunello
pushed a commit
to fercomunello/quarkus
that referenced
this issue
Aug 31, 2022
Adds missing support for classes referenced by `@JsonTypeIdResolver`, which is particularly important when using native builds. The added integration tests are likely only meaningful when run in native mode (`-Pnative`), otherwise "good old Java reflection" does its job. Fixes quarkusio#27346
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
io.quarkus.jackson.deployment.JacksonProcessordetects the classes referenced by e.g.@JsonSerialize,@JsonDeserialize, but it doesn't handle classes referenced by@JsonTypeIdResolverannotations.It seems that "just handling" the value of
JsonTypeIdResolver.value()likeJsonDeserialize.using()should fix this.Could someone quickly cross-check whether my assumption sounds legit? Happy to provide a PR with the described change to fix this issue.
Expected behavior
No response
Actual behavior
No response
How to Reproduce?
No response
Output of
uname -aorverNo response
Output of
java -versionNo response
GraalVM version (if different from Java)
No response
Quarkus version or git rev
No response
Build tool (ie. output of
mvnw --versionorgradlew --version)No response
Additional information
No response
The text was updated successfully, but these errors were encountered: