Update Arquillian support feature to not depend on servlet feature

OpenLiberty / liberty-arquillian

Arquillian Liberty Managed and Remote containers

Apache License 2.0

11 stars 29 forks source link

Update Arquillian support feature to not depend on servlet feature #117

Closed jhanders34 closed 2 years ago

jhanders34 commented 2 years ago

Update to use JMX instead of a servlet to get the Exception data from the server so that we do not need to enable the servlet feature since it is not required by the Core Profile.

Fixes: #113

cherylking commented 2 years ago

@kathrynkodama I want to make sure you review this, because I know you replaced some of the code that used MBeans to use Servlet instead - specifically to getServletNames I believe.

jhanders34 commented 2 years ago

Yeah this PR is stage one, but we definitely are using servlet still with how things are done. It would be ideal to not use a servlet when one isn't specified and be able to use a restful resource instead, but that is going to take some more thought / ideas.

jhanders34 commented 2 years ago

I noticed the builds ran with 21.0.0.4-beta. Might want to bump that up to something more recent to test these changes.

I updated the PR to use 22.0.0.6 instead by default.