I really prefer to make the spec the default behavior, nobody expects a JAR to be scanned without having a beans.xml but a extension normally if you would like to use CDI, but avoid scanning, you add a CDI extension but no beans.xml, thats exactly like MF and Mojarra is implemented
Lets wait for other opinions :) Am Mi., 22. Sept. 2021 um 11:08 Uhr schrieb Romain Manni-Bucau < rmannibu...@gmail.com>: > Think we should prefer our users over the spec for that so I'd be to > support it with a toggle to enable it > (org.apache.webbeans.skipJarWithExtensionScanning=false by default). > If needed we can add a bdascannerservice with it at true (WebSpecScanner). > And in 2.1 we can change 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. 22 sept. 2021 à 11:06, Thomas Andraschko < > andraschko.tho...@gmail.com> a écrit : > > > Hi, > > > > I discussed with Romain about OWB-1298. > > > > Mojarra has NO beans.xml but a CDI Extension, which currently breaks OWB > > as: > > - WebsocketUserManager is scanned > > - WebsocketUserManager is added in the extension > > > > in the specs (12.1) its defined, that: > > > > An archive which: > > > > - > > > > contains a beans.xml file with the bean-discovery-mode of none, or, > > - > > > > contains an extension and no beans.xml file > > > > is NOT a bean archive. > > > > > > Which means that OWB should NOT scan Mojarra and therefore users can use > > Mojarra with OWB, without adding it to the scan exclusions or setting > other > > properties. > > > > If MyFaces wouldnt be in the default exclusions, it would also break MF > and > > maybe other libs. > > > > We should fix it and make it the default behavior. > > > > It should even improve startup as we skip more archives per default. > > > > Romain mentioned that there could be cases, where it could break apps. So > > we should maybe introduce a new config property to enable the old > behavior > > again, which also scans archives with extensions but without beans.xml > > > > this is also related to org.apache.webbeans.scanBeansXmlOnly > > > > WDYT? > > > > > > Best regards, > > > > Thomas > > >
