Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

OSGi Cannot start Jetty with osgi.boot - Configurations add wrong method taken #6011

Closed
maho7791 opened this issue Feb 25, 2021 · 0 comments · Fixed by #6012
Closed

OSGi Cannot start Jetty with osgi.boot - Configurations add wrong method taken #6011

maho7791 opened this issue Feb 25, 2021 · 0 comments · Fixed by #6012
Assignees

Comments

@maho7791
Copy link

Jetty version
10.0.0, 10.0.1

Java version
11
OS type/version
Linux/Windows

Description
Starting Jetty with jetty.home.bundle=org.eclipse.osgi.boot fails while reading xml and setting the configurations.

It tries to set the array of Strings:
[org.eclipse.jetty.webapp.FragmentConfiguration, org.eclipse.jetty.webapp.JettyWebXmlConfiguration, org.eclipse.jetty.webapp.WebXmlConfiguration, org.eclipse.jetty.webapp.WebAppConfiguration, org.eclipse.jetty.webapp.ServletsConfiguration, org.eclipse.jetty.webapp.JspConfiguration, org.eclipse.jetty.webapp.JaasConfiguration, org.eclipse.jetty.webapp.JndiConfiguration, org.eclipse.jetty.plus.webapp.PlusConfiguration, org.eclipse.jetty.plus.webapp.EnvConfiguration, org.eclipse.jetty.webapp.JmxConfiguration, org.eclipse.jetty.osgi.annotations.AnnotationConfiguration, org.eclipse.jetty.websocket.server.config.JettyWebSocketConfiguration, org.eclipse.jetty.websocket.javax.server.config.JavaxWebSocketConfiguration, org.eclipse.jetty.osgi.boot.OSGiWebInfConfiguration, org.eclipse.jetty.osgi.boot.OSGiMetaInfConfiguration]

org.eclipse.jetty.webapp.Configurations#add()

This method existis multiple times in this class. This one was taken from:

org.eclipse.jetty.xml.XmlConfiguration#invokeMethod (709):

method: public boolean java.util.AbstractList.add(java.lang.Object)
object: class org.eclipse.jetty.webapp.Configurations@1
args: see array above

I think the expected method to be called instead, was:

org.eclipse.jetty.webapp.Configurations#add(@name("configClass") String... configClass)

This prevents Jetty from starting in OSGi using org.eclipse.jetty.osgi.boot, with the default configuration.

@janbartel janbartel self-assigned this Feb 25, 2021
janbartel added a commit that referenced this issue Feb 25, 2021
…tty.osgi.boot

Signed-off-by: Jan Bartel <janb@webtide.com>
janbartel added a commit that referenced this issue Mar 3, 2021
…tty.osgi.boot (#6012)

Signed-off-by: Jan Bartel <janb@webtide.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
2 participants