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

DebugListener module in core refers to non-existent org.eclipse.jetty.server.DebugListener #12191

Closed
janbartel opened this issue Aug 22, 2024 · 0 comments · Fixed by #12254
Closed

DebugListener module in core refers to non-existent org.eclipse.jetty.server.DebugListener #12191

janbartel opened this issue Aug 22, 2024 · 0 comments · Fixed by #12254
Assignees
@gregw @janbartel
Labels
Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement

Comments

@janbartel
Copy link
Contributor

Jetty version(s)
jetty-12.0.x

Jetty Environment
core

The debug.mod core module refers to jetty-debug.xml, which refers to org.eclipse.jetty.server.DebugListener that does not exist, although ee specific DebugListener classes do:

find . -name DebugListener.java

./jetty-ee10/jetty-ee10-servlet/src/main/java/org/eclipse/jetty/ee10/servlet/DebugListener.java
./jetty-ee9/jetty-ee9-nested/src/main/java/org/eclipse/jetty/ee9/nested/DebugListener.java
./jetty-ee8/jetty-ee8-nested/target/generated-sources/ee8/org/eclipse/jetty/ee8/nested/DebugListener.java


find . -name "*.xml" | xargs egrep DebugListener

./jetty-core/jetty-server/src/main/config/etc/jetty-debug.xml:<!-- The DebugListener                                               -->
./jetty-core/jetty-server/src/main/config/etc/jetty-debug.xml:  <New id="DebugListener" class="org.eclipse.jetty.server.DebugListener">
./jetty-core/jetty-server/src/main/config/etc/jetty-debug.xml:  <Call name="addBean"><Arg><Ref refid="DebugListener"/></Arg></Call>
./jetty-core/jetty-server/src/main/config/etc/jetty-debug.xml:        <New class="org.eclipse.jetty.deploy.bindings.DebugListenerBinding">
./jetty-core/jetty-server/src/main/config/etc/jetty-debug.xml:          <Arg><Ref refid="DebugListener"/></Arg>


find . -name debug.mod

./jetty-core/jetty-server/src/main/config/modules/debug.mod
@janbartel janbartel added the Bug For general bugs on Jetty side label Aug 22, 2024
@janbartel janbartel added the Sponsored This issue affects a user with a commercial support agreement label Aug 22, 2024
@sbordet sbordet moved this to 🏗 In progress in Jetty 12.0.13 - FROZEN Aug 28, 2024
janbartel added a commit that referenced this issue Sep 3, 2024
@janbartel
Issue #12191 reinstate debug listener modules
e2a3eea
@janbartel janbartel mentioned this issue Sep 3, 2024
Closed
@joakime joakime mentioned this issue Sep 25, 2024
Merged
@joakime joakime moved this to 🏗 In progress in Jetty 12.0.14 Sep 25, 2024
janbartel added a commit that referenced this issue Sep 30, 2024
@janbartel @gregw
Jetty 12.0.x 12191 alt debug listener (#12254)
97ff548 
* Issue #12191 reinstate debug listener modules

---------

Co-authored-by: Greg Wilkins <[email protected]>
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Jetty 12.0.14 Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gregw gregw

@janbartel janbartel

Labels
Bug For general bugs on Jetty side Sponsored This issue affects a user with a commercial support agreement
Projects
No open projects
Jetty 12.0.14
Status: ✅ Done
Milestone
No milestone
2 participants
@gregw @janbartel