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

Clean up anonymous class loader in Undertow setup #43727

Open
dmlloyd opened this issue Oct 5, 2024 · 0 comments · May be fixed by #43728
Open

Clean up anonymous class loader in Undertow setup #43727

dmlloyd opened this issue Oct 5, 2024 · 0 comments · May be fixed by #43728
Labels
area/housekeeping Issue type for generalized tasks not related to bugs or enhancements

Comments

@dmlloyd
Copy link
Member

dmlloyd commented Oct 5, 2024

Description

The setup code in the Undertow recorder sets the CL twice, and then finally chooses an anonymous subclass of ClassLoader if there is no TCCL set.

Implementation ideas

No response
@dmlloyd dmlloyd added the area/housekeeping Issue type for generalized tasks not related to bugs or enhancements label Oct 5, 2024
@dmlloyd dmlloyd linked a pull request Oct 5, 2024 that will close this issue
Open
dmlloyd added a commit to dmlloyd/quarkus that referenced this issue Oct 5, 2024
@dmlloyd
Do not set class loader twice in Undertow setup
8eee52a 
Also use the system (app) class loader instead of an anonymous, empty class loader which delegates to the system class loader

Fixes quarkusio#43727
@dmlloyd dmlloyd mentioned this issue Oct 5, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/housekeeping Issue type for generalized tasks not related to bugs or enhancements
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Do not set class loader twice in Undertow setup dmlloyd/quarkus
1 participant
@dmlloyd