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

Logback configuration that relies on inner-classes does not work in a native image #44021

Closed
wants to merge 1 commit into from

Conversation

nosan
Copy link
Contributor

@nosan nosan commented Jan 30, 2025

Before this commit, the generated name for the inner class had the wrong format <package>.<parent>.<child> (canonical name). GraalVM expects $ to separate the parent from the inner class.

This commit updates SpringBootJoranConfigurator to generate an appropriate format for a class name. Specifically, an inner class should be separated by a dollar sign, not a dot.

See gh-44016

@nosan nosan changed the base branch from main to 3.4.x January 30, 2025 09:22
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 30, 2025
@nosan nosan force-pushed the 44016 branch 2 times, most recently from 2518f9f to 9935ac3 Compare January 30, 2025 09:25
Before this commit, the generated name for the inner class had the
wrong format <package>.<parent>.<child> (canonical name).
GraalVM expects $ to separate the parent from the inner class.

This commit updates SpringBootJoranConfigurator to generate
an appropriate format for a class name. Specifically, an inner class
should be separated by a dollar sign, not a dot.

See spring-projectsgh-44016

Signed-off-by: Dmytro Nosan <dimanosan@gmail.com>
@wilkinsona wilkinsona changed the title Update SpringBootJoranConfigurator to use proper GraalVM format Logback configuration that relies on inner-classes does not work in a native image Jan 30, 2025
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 30, 2025
@wilkinsona wilkinsona added this to the 3.3.x milestone Jan 30, 2025
@wilkinsona wilkinsona self-assigned this Jan 30, 2025
@nosan
Copy link
Contributor Author

nosan commented Jan 30, 2025

Oh, I apologize for the mistake. The target branch should be 3.3.x. :(

@wilkinsona
Copy link
Member

wilkinsona commented Jan 30, 2025

No problem at all! You've done more than enough already. I'll take care of it as part of merging the fix.

@wilkinsona wilkinsona modified the milestones: 3.3.x, 3.3.9 Jan 30, 2025
wilkinsona pushed a commit that referenced this pull request Jan 30, 2025
Before this commit, the generated name for the inner class had the
wrong format <package>.<parent>.<child> (canonical name).
GraalVM expects $ to separate the parent from the inner class.

This commit updates SpringBootJoranConfigurator to generate
an appropriate format for a class name. Specifically, an inner class
should be separated by a dollar sign, not a dot.

See gh-44021

Signed-off-by: Dmytro Nosan <dimanosan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants