Results 10 comments of Gary Tully

peek at https://github.com/apache/activemq/blob/138e52b08c2f49b730817932a6e63f2a135854f1/activemq-unit-tests/src/test/java/org/apache/activemq/network/DuplexNetworkMBeanTest.java for some unit test inspiration, or other tests in that package

looks like there is a v4.0.4 with a fix. https://github.com/fabric8io/kubernetes-client/commit/05f5c4f91ca5691691a56fddd685486e7ec644b6

make me wonder what is actually being verified, at the point of clientsession start, it is not possible to know the state of the broker unless we wait for some...

any thoughts in why this shows up now? that pattern is in a bunch of tests but from what I can see, it fails b/c the receiveImmediate happens before all...

Using a thread local to propagate the session subject is fine, but it needs to be scoped to the user of that thread for the request, and cleared on response....

I don't know that it helps, the values in question come from configuration. We have no choice but to trust configuration, i.e: the file system, where our sources live. These...

I have opened https://issues.apache.org/jira/browse/ARTEMIS-4766 to follow up.

> I agree w/ @cshannon here. There should be a setting to support honoring a valid list of package names -- there could even be an out-of-the-box default end-users could...

> Looks good, have you run the full test suite on this set of changes? yes!