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

Jetty 9.4.x reports jetty.home system variable instead of jetty.home.bundle when it can't find jetty.home.bundle #12668

Open
sdybiec opened this issue Jan 3, 2025 · 1 comment
Assignees
Labels
Bug For general bugs on Jetty side

Comments

@sdybiec
Copy link

sdybiec commented Jan 3, 2025

Jetty version(s)

v9.4.56

Java version/vendor (use: java -version)

Java(TM) SE Runtime Environment (build 17.0.5+9-LTS-191)
Java HotSpot(TM) 64-Bit Server VM (build 17.0.5+9-LTS-191, mixed mode, sharing)

OS type/version

Windows 10

Description
When Jetty can't resolve the OSGi bundle specified by the jetty.home.bundle system variable, it reports a warning with the value of the jetty.home system variable instead of jetty.home.bundle system variable.

Current line 197: LOG.warn("Unable to find the jetty.home.bundle named {}", jettyHomeSysProp);
Proposed line 197: LOG.warn("Unable to find the jetty.home.bundle named {}", jettyHomeBundleSysProp);

LOG.warn("Unable to find the jetty.home.bundle named {}", jettyHomeSysProp);

How to reproduce?

Set jetty.home.bundle to a non-existent bundle.

@sdybiec sdybiec added the Bug For general bugs on Jetty side label Jan 3, 2025
@janbartel janbartel self-assigned this Jan 3, 2025
@janbartel
Copy link
Contributor

Jetty 9.4 is at end of life, however, I can see that the same wrong property is used in jetty-12.0.x, so PR #12677 fixes that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For general bugs on Jetty side
Projects
None yet
Development

No branches or pull requests

2 participants