RE: [Newsletter] Re: [Newsletter] Re: Missing Guava in TomEE 7.0.4 Plus?

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

RE: [Newsletter] Re: [Newsletter] Re: Missing Guava in TomEE 7.0.4 Plus?

fabian-a.richter
What do you mean with "leak"? We do not have it in our webapp, we only have guava but it is not picked up by the parent classloader :(

-----Original Message-----
From: Romain Manni-Bucau <[hidden email]>
Sent: Wednesday, June 20, 2018 3:00 PM
To: [hidden email]
Subject: *EXT* [Newsletter] Re: [Newsletter] Re: Missing Guava in TomEE 7.0.4 Plus?

Hmm if we provide it OOTB that's a "leak" on our side, if you have it in the webapp just drop it.

Romain Manni-Bucau
@rmannibucau <https://twitter.com/rmannibucau> |  Blog <https://rmannibucau.metawerx.net/> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book <https://www.packtpub.com/application-development/java-ee-8-high-performance>


Le mer. 20 juin 2018 à 14:58, <[hidden email]> a écrit :

> Is there a way to not "import shibboleth"? I am pretty sure we don’t
> need it...
>
> -----Original Message-----
> From: Romain Manni-Bucau <[hidden email]>
> Sent: Wednesday, June 20, 2018 2:49 PM
> To: [hidden email]
> Subject: *EXT* [Newsletter] Re: Missing Guava in TomEE 7.0.4 Plus?
>
> Hi,
>
> yes and no. We intentionnally drop guava and it works with opensaml
> until you import shibboleth which requires it. At that moment you have
> to either drop opensaml libs from tomee libs or add the missing ones.
>
> Romain Manni-Bucau
> @rmannibucau <https://twitter.com/rmannibucau> |  Blog <
> https://rmannibucau.metawerx.net/> | Old Blog <
> http://rmannibucau.wordpress.com> | Github
> <https://github.com/rmannibucau>
> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book <
> https://www.packtpub.com/application-development/java-ee-8-high-perfor
> mance
> >
>
>
> Le mer. 20 juin 2018 à 14:46, <[hidden email]> a
> écrit :
>
> > Hey,
> >
> >
> >
> > when I try to initialize OpenSAML in TomEE 7.0.4 with
> > InitializationService.initialize(); I get
> >
> >
> >
> > java.lang.ClassNotFoundException: com.google.common.base.Function
> >
> >            
> > java.net.URLClassLoader.findClass(URLClassLoader.java:381)
> >
> >             java.lang.ClassLoader.loadClass(ClassLoader.java:424)
> >
> >             java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> >
> >
> > net.shibboleth.utilities.java.support.xml.AttributeSupport.getAttrib
> > ut
> > eValueAsQName(AttributeSupport.java:334)
> >
> >
> > org.opensaml.core.xml.config.XMLConfigurator.initializeObjectProvide
> > rs
> > (XMLConfigurator.java:231)
> >
> >
> > org.opensaml.core.xml.config.XMLConfigurator.load(XMLConfigurator.ja
> > va
> > :203)
> >
> >
> > org.opensaml.core.xml.config.XMLConfigurator.load(XMLConfigurator.ja
> > va
> > :188)
> >
> >
> > org.opensaml.core.xml.config.XMLConfigurator.load(XMLConfigurator.ja
> > va
> > :162)
> >
> >
> > org.opensaml.core.xml.config.AbstractXMLObjectProviderInitializer.in
> > it
> > (AbstractXMLObjectProviderInitializer.java:52)
> >
> >
> > org.opensaml.core.xml.config.XMLObjectProviderInitializer.init(XMLOb
> > je
> > ctProviderInitializer.java:45)
> >
> >
> > org.opensaml.core.config.InitializationService.initialize(Initializa
> > ti
> > onService.java:56)
> >
> >
> >
> > As soon as I add guava-18.0.jar to TomEE/lib it works.
> >
> >
> >
> > Did you guys miss to package this transitive dependency of opensaml
> > (via java-support)?
> >
> >
> >
> > Thanks and best
> >
> > Fabian
> >
>

smime.p7s (7K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [Newsletter] Re: [Newsletter] Re: Missing Guava in TomEE 7.0.4 Plus?

Romain Manni-Bucau
It is not in tomee lib so should be coming from your app or environment :s

by leak i meant we don't intend to deliver this lib out of the box

Romain Manni-Bucau
@rmannibucau <https://twitter.com/rmannibucau> |  Blog
<https://rmannibucau.metawerx.net/> | Old Blog
<http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> |
LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book
<https://www.packtpub.com/application-development/java-ee-8-high-performance>


Le mer. 20 juin 2018 à 15:06, <[hidden email]> a écrit :

> What do you mean with "leak"? We do not have it in our webapp, we only
> have guava but it is not picked up by the parent classloader :(
>
> -----Original Message-----
> From: Romain Manni-Bucau <[hidden email]>
> Sent: Wednesday, June 20, 2018 3:00 PM
> To: [hidden email]
> Subject: *EXT* [Newsletter] Re: [Newsletter] Re: Missing Guava in TomEE
> 7.0.4 Plus?
>
> Hmm if we provide it OOTB that's a "leak" on our side, if you have it in
> the webapp just drop it.
>
> Romain Manni-Bucau
> @rmannibucau <https://twitter.com/rmannibucau> |  Blog <
> https://rmannibucau.metawerx.net/> | Old Blog <
> http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau>
> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book <
> https://www.packtpub.com/application-development/java-ee-8-high-performance
> >
>
>
> Le mer. 20 juin 2018 à 14:58, <[hidden email]> a
> écrit :
>
> > Is there a way to not "import shibboleth"? I am pretty sure we don’t
> > need it...
> >
> > -----Original Message-----
> > From: Romain Manni-Bucau <[hidden email]>
> > Sent: Wednesday, June 20, 2018 2:49 PM
> > To: [hidden email]
> > Subject: *EXT* [Newsletter] Re: Missing Guava in TomEE 7.0.4 Plus?
> >
> > Hi,
> >
> > yes and no. We intentionnally drop guava and it works with opensaml
> > until you import shibboleth which requires it. At that moment you have
> > to either drop opensaml libs from tomee libs or add the missing ones.
> >
> > Romain Manni-Bucau
> > @rmannibucau <https://twitter.com/rmannibucau> |  Blog <
> > https://rmannibucau.metawerx.net/> | Old Blog <
> > http://rmannibucau.wordpress.com> | Github
> > <https://github.com/rmannibucau>
> > | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book <
> > https://www.packtpub.com/application-development/java-ee-8-high-perfor
> > mance
> > >
> >
> >
> > Le mer. 20 juin 2018 à 14:46, <[hidden email]> a
> > écrit :
> >
> > > Hey,
> > >
> > >
> > >
> > > when I try to initialize OpenSAML in TomEE 7.0.4 with
> > > InitializationService.initialize(); I get
> > >
> > >
> > >
> > > java.lang.ClassNotFoundException: com.google.common.base.Function
> > >
> > >
> > > java.net.URLClassLoader.findClass(URLClassLoader.java:381)
> > >
> > >             java.lang.ClassLoader.loadClass(ClassLoader.java:424)
> > >
> > >             java.lang.ClassLoader.loadClass(ClassLoader.java:357)
> > >
> > >
> > > net.shibboleth.utilities.java.support.xml.AttributeSupport.getAttrib
> > > ut
> > > eValueAsQName(AttributeSupport.java:334)
> > >
> > >
> > > org.opensaml.core.xml.config.XMLConfigurator.initializeObjectProvide
> > > rs
> > > (XMLConfigurator.java:231)
> > >
> > >
> > > org.opensaml.core.xml.config.XMLConfigurator.load(XMLConfigurator.ja
> > > va
> > > :203)
> > >
> > >
> > > org.opensaml.core.xml.config.XMLConfigurator.load(XMLConfigurator.ja
> > > va
> > > :188)
> > >
> > >
> > > org.opensaml.core.xml.config.XMLConfigurator.load(XMLConfigurator.ja
> > > va
> > > :162)
> > >
> > >
> > > org.opensaml.core.xml.config.AbstractXMLObjectProviderInitializer.in
> > > it
> > > (AbstractXMLObjectProviderInitializer.java:52)
> > >
> > >
> > > org.opensaml.core.xml.config.XMLObjectProviderInitializer.init(XMLOb
> > > je
> > > ctProviderInitializer.java:45)
> > >
> > >
> > > org.opensaml.core.config.InitializationService.initialize(Initializa
> > > ti
> > > onService.java:56)
> > >
> > >
> > >
> > > As soon as I add guava-18.0.jar to TomEE/lib it works.
> > >
> > >
> > >
> > > Did you guys miss to package this transitive dependency of opensaml
> > > (via java-support)?
> > >
> > >
> > >
> > > Thanks and best
> > >
> > > Fabian
> > >
> >
>