Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Jan 31 15:34
    manovotn commented #1897
  • Jan 31 15:33
    manovotn commented #1899
  • Jan 31 15:33

    manovotn on master

    Upgrade SpotBugs versions, Supp… (compare)

  • Jan 31 15:33
    manovotn closed #1898
  • Jan 31 14:41
    nziakova commented #1899
  • Jan 31 14:40
    nziakova commented #1899
  • Jan 31 14:18
    nziakova commented #1899
  • Jan 31 14:12
    nziakova review_requested #1899
  • Jan 31 14:12
    nziakova opened #1899
  • Jan 31 11:18
    JavaTesting1 commented #64
  • Jan 31 10:20
    manovotn review_requested #1898
  • Jan 31 10:20
    manovotn opened #1898
  • Jan 31 09:14

    manovotn on master

    WELD-2517: Weld SE - fix Securi… (compare)

  • Jan 31 09:14
    manovotn closed #1895
  • Jan 31 09:01
    nziakova edited #1897
  • Jan 31 09:01
    nziakova edited #1897
  • Jan 31 09:00
    nziakova review_requested #1897
  • Jan 31 08:58
    nziakova opened #1897
  • Jan 30 12:51
    manovotn review_requested #1896
  • Jan 30 12:51
    manovotn review_requested #1896
Svante Kumlien
@kumlien
Hi @manovotn , have a look at this one: https://github.com/kumlien/async-events
@manovotn Would be interesting to hear what you think about it.
Matej Novotny
@manovotn
Hey @kumlien
thanks, i wil ltake a look
hmm, gradle rookie here, how do I build it? Just gradle build or something more?
also, I can see you are running JDK 11, right?
Svante Kumlien
@kumlien
@manovotn gradle build should hopefully do it and yes jdk11
Matej Novotny
@manovotn
out of curiosity, have you tried that with JDK 8?
just to eliminate any funky stuff hapenning on newer JDKs
Svante Kumlien
@kumlien
No, and I don't think I have any jdk11 specific stuff in it so you can try to build it with jdk8
Matej Novotny
@manovotn
gradle wont let me
Svante Kumlien
@kumlien
hang on...
you can change the build.gradle:
sourceCompatibility = JavaVersion.VERSION_1_8
targetCompatibility = JavaVersion.VERSION_1_8
Matej Novotny
@manovotn
okay, I will play around with it a bit
Svante Kumlien
@kumlien
ok!
Matej Novotny
@manovotn
@kumlien I am seeing all events delivered on WFLY 16, server logs look something like this: https://pastebin.com/gaqycJkV
and that^ is true for both JDK versions
Svante Kumlien
@kumlien
ok. Are you running your wildfly on jdk11 (I am).
Matej Novotny
@manovotn
openjdk version "11.0.2" 2019-01-15
Svante Kumlien
@kumlien
hmm, that is weird
I'm using openjdk version "11.0.1" 2018-10-16 LTS
Matej Novotny
@manovotn
that would hardly make a difference
Svante Kumlien
@kumlien
shouldn't matter
Matej Novotny
@manovotn
aren't you somehow shutting down the app or the server preliminary? E.g. right after getting "OK" reponse? Because that precedes actual notifications in your app
just shooting blind here cause I don't really know what might be the problem
Svante Kumlien
@kumlien
Well, when I debug the server and I can see that it throws on line 60 in RegisteBuiltin:
en = Thread.currentThread().getContextClassLoader().getResources("META-INF/services/" + Providers.class.getName());
with a NP
(should be RegisterBuiltin)
Matej Novotny
@manovotn
Hold on, I might have something
my WFLY might not have been pristine version, I think I patched it previously with newer Weld version while testing someting else
right, now I can actually see what you see
sorry for that, might fault
Svante Kumlien
@kumlien
ahh
makes me happy!
Matej Novotny
@manovotn
but that can also mean that your problem is actually already fixed in master ;)
lemme try that
Svante Kumlien
@kumlien
great!
If I catch the exception in my listener I get this stacktrace: https://pastebin.com/zVcxd6ru
Matej Novotny
@manovotn
ah well that explains it
thats going to be weld/core@88f5cd1
aka https://issues.jboss.org/browse/WELD-2573 and the issues linked to it on WFLY side
that was problem with TCCL not being preserved and resteasy relied on it
Svante Kumlien
@kumlien
great!
Matej Novotny
@manovotn
and voila, putting latest Weld master into WFLY, your sample works
Svante Kumlien
@kumlien
magic :)
will the fix go into a wildfly 16.x release soon?
Matej Novotny
@manovotn
glad we nailed it down!
@kumlien there is going to be Weld 3.1.1.Final release next week if all goes as planned (you can see that in the JIRA^)
sometime after that there is going to be new WFLY using it, WFLY 17
if that doesn't suit you, with every release we tend to also publish a patch file for the latest released WFLY, so in this case there should be a patch for WFLY 16 that will put Weld 3.1.1 into it
Svante Kumlien
@kumlien
nice!
let's close this one then and Užij si víkend
Matej Novotny
@manovotn
hehe, thanks, likaledes
Matej Novotny
@manovotn
@kumlien Hello, just an FYI, Weld 3.1.1.Final is now live. So feel free to test it out, link to the patch and notes on how to patch WFLY should all be available here - http://weld.cdi-spec.org/news/2019/05/07/weld-311Final/