Skip to content

GH-8613: Add JsonPropertyAccessor type for native #8614

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

Merged
merged 1 commit into from
May 8, 2023

Conversation

artembilan
Copy link
Member

Fixes #8613

If JsonPropertyAccessor is registered for SpEL, it would be great to have it working in native images as well.

Since SpEL is fully based on reflection, expose
JsonPropertyAccessor$ComparableJsonNode and JsonPropertyAccessor$ArrayNodeAsList reflection hints for their method invocations from SpEL

Cherry-pick to 6.0.x

Fixes spring-projects#8613

If `JsonPropertyAccessor` is registered for SpEL, it would be great
to have it working in native images as well.

Since SpEL is fully based on reflection, expose
`JsonPropertyAccessor$ComparableJsonNode` and `JsonPropertyAccessor$ArrayNodeAsList`
reflection hints for their method invocations from SpEL

**Cherry-pick to `6.0.x`**
@garyrussell garyrussell merged commit ffe50d2 into spring-projects:main May 8, 2023
garyrussell pushed a commit that referenced this pull request May 8, 2023
Fixes #8613

If `JsonPropertyAccessor` is registered for SpEL, it would be great
to have it working in native images as well.

Since SpEL is fully based on reflection, expose
`JsonPropertyAccessor$ComparableJsonNode` and `JsonPropertyAccessor$ArrayNodeAsList`
reflection hints for their method invocations from SpEL

**Cherry-pick to `6.0.x`**
@garyrussell
Copy link
Contributor

...and cherry-picked as 5278a38

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.

Add GraalVM reflection config for JsonPropertyAccessor
2 participants