Brad Maxwell
Brad Maxwell
@tadamski can you review this one?
This includes tests for it and previous issue, WFLY-16567 is already merged. It needs component upgrades for WF Http Client, looks like WFLY-16741 is merged. And ejb client is merged...
@chengfang thanks, I looked but it did not make sense how it could return null on the ejb lookup and the test had passed on my machine previously, so not...
@emmartins can you review this PR? It has missed making it into a couple of CPs
@TomasHofman thanks, I've updated my PR, you can use it or yours which ever. I reviewed your changes where the one test was returning null and then the clean up...
@markt-asf can you check this to see if it can be merged, this issue was addressed in Tomcat https://bz.apache.org/bugzilla/show_bug.cgi?id=57583 https://github.com/apache/tomcat80/commit/63665340bb6fb123d93627cf4eac62dadd1545ab https://github.com/apache/tomcat80/commit/b69ed7e5f1bfd20ac09f8ea70e77731e1f1fcb44
@arjantijms Is this PR good to merge?