Thats true Romain... We have some code where we need to check if we are running in an http request and not e.g. in a background worker or on startup, to determine if we can take some settings from the user session.
As i said, it's no problem if you are checking RequestScoped && SessionScoped. RequestScoepd is not active and therefor the lazyStartSessionContext will not happen. so +0, it was just an idea :) 2016-03-02 14:20 GMT+01:00 Romain Manni-Bucau <[email protected]>: > About the log: originally it was not there and it is a nightmare to > understand OWB is not well setup in your container. The log shouldn't > happear - as much as you shouldn't even try to use session scope in a > thread where it is not available theorically. A consensus can be to log it > once but in "big" (like taking 20 lines of logs just to say something is > not correct). A step further can be to return a fake context never active > in such cases. > > Wydt? > > > Romain Manni-Bucau > @rmannibucau <https://twitter.com/rmannibucau> | Blog > <http://rmannibucau.wordpress.com> | Github > <https://github.com/rmannibucau> | LinkedIn > <https://www.linkedin.com/in/rmannibucau> | Tomitriber > <http://www.tomitribe.com> > > 2016-03-02 13:56 GMT+01:00 Thomas Andraschko <[email protected]> > : > >> The method in deltaspike is ok, it just checks if the context is active >> in the current thread. >> >> +1 for jira and a patch. >> >> 2016-03-02 13:47 GMT+01:00 Karl Kildén <[email protected]>: >> >>> Right, Deltaspike hides the ugly that you call a boolean method but >>> instead of false you get ContextNotActiveException. One of the more >>> mystical parts of the specification if you ask me. Maybe you are refereeing >>> to how the BeanManager should behave if the thread is not managed? >>> >>> It is simply a little bit over my head but at least nothing in the >>> javadoc for ContextUtils in deltaspike forbids usage from unmanaged threads >>> and frankly if that was a requirement it seems like a poor idea to offer >>> such static methods in the first place since managed threads have >>> injectionpoints. >>> >>> Regarding a JIRA och a patch: I will take care of it. >>> >>> Thanks for helping me understanding this better >>> >>> cheers / Kalle >>> >> >> >
