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.

Sign up for GitHub

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

Jump to bottom

Fixing Java Configurations #365

Open
pturchinetz opened this issue May 26, 2022 · 3 comments · May be fixed by #548
Open

Fixing Java Configurations #365

pturchinetz opened this issue May 26, 2022 · 3 comments · May be fixed by #548
Assignees
@tyBouch

Comments

@pturchinetz
Copy link
Collaborator

14:53:57.058 ERROR exceptions.FileException (FileException.java:158) - C:\SOURCE\sentinel-helicoid\Helicoid\conf\sentinel.yml Could not load the timeout property because of the exception: Unable to make field private final java.lang.String java.io.File.path accessible: module java.base does not "opens java.io" to unnamed module @3a4afd8d.
Please fix the file or pass the property in on the commandline using the -Dtimeout= option.
14:53:59.077 WARN configurations.Configuration (Configuration.java:256) - localhost env being used by default. No env property set. This can be set in the sentinel.yml config file with a 'env=' property or on the command line with the switch '-Denv='.

We are unable to load config files because of this issue
@pturchinetz
Copy link
Collaborator Author

"JDK newer than 15 fails due to this"

@tyBouch
Copy link
Collaborator

tyBouch commented Apr 19, 2023 •
edited
Loading

Issue still occurring. Same error.

Could not load the timeout property because of the exception: Unable to make field private final java.lang.String java.io.File.path accessible: module java.base does not "opens java.io" to unnamed module

Different @ text at the end.

@tyBouch
Copy link
Collaborator

tyBouch commented May 4, 2023

Root cause seems to be the ConfigurationData class extending File. This seems to be no longer possible. The error appears to have been the File class throwing when it was trying to make portions of itself exposed to ConfigurationData.

I have a fix coming which will refactor the class to utilize File instead of extending it.

@tyBouch tyBouch self-assigned this May 4, 2023
@tyBouch tyBouch linked a pull request May 4, 2023 that will close this issue
365 fix configuration file reading #548
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tyBouch tyBouch

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

365 fix configuration file reading dougnoel/sentinel
2 participants
@tyBouch @pturchinetz