Skip to content

Bael 7805 how to fix json io exception failed making field property accessible in gson - #19293

Open
MBuczkowski2025 wants to merge 14 commits into
eugenp:masterfrom
MBuczkowski2025:BAEL-7805_How_to_fix_JsonIOException_Failed_making_field_property_accessible_in_Gson
Open

Bael 7805 how to fix json io exception failed making field property accessible in gson#19293
MBuczkowski2025 wants to merge 14 commits into
eugenp:masterfrom
MBuczkowski2025:BAEL-7805_How_to_fix_JsonIOException_Failed_making_field_property_accessible_in_Gson

Conversation

@MBuczkowski2025

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread json-modules/gson-4/pom.xml Outdated
Comment thread json-modules/gson-4/pom.xml Outdated
Comment on lines +24 to +27
<maven.compiler.source>17</maven.compiler.source>
<maven.compiler.target>17</maven.compiler.target>
<maven.compiler.release>17</maven.compiler.release>
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similar for these, can we rely on the parent for them?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without this setting, the build fails with a 'records are not supported in -source 8' error. Anyway, this fragment is shown in the tutorial. I'd leave it as it is.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean, without the Java 17 setting. Encoding is removed

Comment thread json-modules/gson-4/gson-module/src/test/java/gson/exception/ModularGsonTest.java Outdated
Comment thread json-modules/gson-4/gson-module/pom.xml Outdated
Comment thread json-modules/gson-4/pom.xml Outdated
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.

2 participants