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

Fields that are not exported by their module cause the parent object to fail #678

Closed
MrMineO5 opened this issue Sep 25, 2022 · 3 comments · Fixed by sourceplusplus/probe-jvm#140
Assignees
Labels
a:bug Something isn't working in:probe-jvm of:medium-effort Requires a maximum of a few days
Milestone

Comments

@MrMineO5
Copy link
Contributor

When the JVM probe generates debug information for objects, it reads their fields. If one of those fields is not exported, the entire object is marked with EXCEPTION_OCCURRED rather than just that field.
Unable to make field sun.net.httpserver.ExchangeImpl sun.net.httpserver.HttpExchangeImpl.impl accessible: module jdk.httpserver does not "opens sun.net.httpserver" to unnamed module @2da3b078

@BFergerson BFergerson added a:bug Something isn't working in:probe-jvm labels Sep 25, 2022
@BFergerson BFergerson self-assigned this Sep 25, 2022
@BFergerson BFergerson added this to the v0.6.10 milestone Sep 25, 2022
@BFergerson
Copy link
Member

prerequisite: #552

@MrMineO5
Copy link
Contributor Author

MrMineO5 commented Sep 27, 2022

Related to: google/gson#1979

Creating a TypeAdapterFactory for JVM internals and adding it to ModelSerializer.kt should resolve the issue, though it may prove difficult to maintain

@BFergerson BFergerson modified the milestones: v0.7.0, v0.7.1 Sep 27, 2022
@BFergerson BFergerson assigned MrMineO5 and unassigned BFergerson Sep 28, 2022
@BFergerson BFergerson modified the milestones: v0.7.1, v0.7.2 Oct 1, 2022
@BFergerson BFergerson linked a pull request Oct 9, 2022 that will close this issue
@BFergerson
Copy link
Member

BFergerson commented Oct 9, 2022

Spent a bit of time trying to figure this issue out. I have a basic working example that needs to be cleaned up. Taking a break from this issue and will look to have it done for the next version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:bug Something isn't working in:probe-jvm of:medium-effort Requires a maximum of a few days
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

2 participants