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

fix(java17): add Jackson converter to RestAdapters to avoid Gson #1174

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

mattgogerly
Copy link
Member

@mattgogerly mattgogerly commented Sep 20, 2023

Fixes spinnaker/spinnaker#6881

If no converter is set for a RestAdapter it defaults to Gson, which uses reflection to set fields. JRE 17 says no bueno to that on classes like Instant that are now making fields inaccessible.

@mattgogerly
Copy link
Member Author

@Mergifyio update

@mergify
Copy link
Contributor

mergify bot commented Sep 20, 2023

update

✅ Branch has been successfully updated

@mattgogerly mattgogerly added the ready to merge Approved and ready for merge label Sep 20, 2023
@mergify mergify bot added the auto merged label Sep 20, 2023
@mergify mergify bot merged commit c1bd0b1 into master Sep 20, 2023
5 checks passed
@mattgogerly mattgogerly deleted the fix/jre17-retrofit branch September 20, 2023 15:18
@mattgogerly
Copy link
Member Author

@Mergifyio backport release-1.32.x

@mergify
Copy link
Contributor

mergify bot commented Sep 20, 2023

backport release-1.32.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Sep 20, 2023
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit c1bd0b1)

# Conflicts:
#	igor-web/src/main/groovy/com/netflix/spinnaker/igor/config/PluginMonitorConfig.java
mattgogerly added a commit that referenced this pull request Sep 20, 2023
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
mattgogerly added a commit that referenced this pull request Sep 20, 2023
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
mattgogerly added a commit that referenced this pull request Sep 20, 2023
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
mergify bot added a commit that referenced this pull request Sep 20, 2023
…) (#1176)

Co-authored-by: Matt Gogerly <6519811+mattgogerly@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged ready to merge Approved and ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Igor Java17 Slim Image fails to start
2 participants