Skip to content

Fixes after rebase#133

Merged
vpelikh merged 3 commits into
masterfrom
fix/modeldeserializer-stringnode-cast
Jul 24, 2026
Merged

Fixes after rebase#133
vpelikh merged 3 commits into
masterfrom
fix/modeldeserializer-stringnode-cast

Conversation

@vpelikh

@vpelikh vpelikh commented Jul 24, 2026

Copy link
Copy Markdown
Owner

No description provided.

@vpelikh
vpelikh force-pushed the fix/modeldeserializer-stringnode-cast branch from 5b7db9e to 07d77a9 Compare July 24, 2026 07:15
vpelikh added 3 commits July 24, 2026 10:18
…eption contract

The refactor in f3de17d replaced (StringNode) typeNode .textValue() with typeNode.stringValue(). In Jackson 3, BaseJsonNode.stringValue() throws UnsupportedOperationException, not ClassCastException.

- deserializeSchemaWithType: Restore (StringNode) cast so non-string type values throw ClassCastException as expected
- getNodeAsString: Same fix for format field
- getRefAsString: instanceof StringNode check with isValueNode() fallback to handle non-text $ref values
Ticket5126Test: com.fasterxml.jackson.databind -> tools.jackson.databind
Ticket5185Test: javax.validation -> jakarta.validation
The SWAGGER_JAXRS2_DEPENDENCY constant and the build.gradle templates still referenced 'io.swagger.core.v3:swagger-jaxrs2' via concatenation, producing an invalid Gradle notation: io.github.vpelikh:swagger-jakarta-rest:io.swagger.core.v3:...:4.0.1-SNAPSHOT

Consolidate the full fork coordinate into the constant and use direct interpolation in all 4 build.gradle templates.
@vpelikh
vpelikh force-pushed the fix/modeldeserializer-stringnode-cast branch from 07d77a9 to b77255d Compare July 24, 2026 07:18
@vpelikh vpelikh changed the title fix: restore StringNode cast in ModelDeserializer to preserve ClassCa… Fixes after rebase Jul 24, 2026
@vpelikh
vpelikh merged commit da459f7 into master Jul 24, 2026
5 checks passed
@vpelikh
vpelikh deleted the fix/modeldeserializer-stringnode-cast branch July 24, 2026 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant