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: unwrap lambda deserialization cause #136

Merged
merged 2 commits into from
Apr 23, 2024

Conversation

mcollovati
Copy link
Contributor

Based on the JDK version, SerializedLambda.readResolve may differently wrap the cause of error.
This change analyzes the exception to unwrap the root cause of the failure, allowing the debug tool to provide the trace of potential unserializable lambda candidates.

References vaadin/flow-components#6256

@mcollovati mcollovati requested a review from tamasmak April 23, 2024 06:38
Base on the JDK version, SerializedLambda.readResolve may differently wrap
the cause of error.
This change analyzes the exception to unwrap the root cause of the failure,
allowing the debug tool to provide the trace of potential unserializable
lambda candidates.

References #135
@mcollovati mcollovati force-pushed the fix/unwrap_classcastexception branch from 9acc7ed to 5fc5a04 Compare April 23, 2024 06:52
@tamasmak tamasmak merged commit f87fdb1 into main Apr 23, 2024
2 checks passed
@tamasmak tamasmak deleted the fix/unwrap_classcastexception branch April 23, 2024 08:13
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