ARTEMIS-3811 Cluster connections clashing with ANYCast addresses

This is a test fix for AMQPClusterReplicaTest

NO-JIRA clarify default for send-to-dla-on-no-route

ARTEMIS-3810 Do not throw error resuming already active XA Transaction

NO-JIRA stopping object on finalize on openWire test

trying to avoid a few test failure seen on CI

ARTEMIS-3807 Simplify Redistributor avoid races, and fix ProtocolMessageLoadBalancingTest

The control existing in Redistributor is not needed as the Queue::deliver will already have a control on re-scheduling the loop and avoid holding references for too long.

ARTEMIS-3806 upgrade logging dependencies

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release 2.22.0

NO-JIRA fixing openTracingPluginTest

ARTEMIS-3799 Fix default ping command pipe error for Windows

ARTEMIS-3803 Fix first NetworkHealthCheck execution

The NetworkHealthCheck hasn't to execute the default command if a custom
command is defined.

ARTEMIS-3805 producer-window-size default

ARTEMIS-3802 Avoiding races on deleteWithID and Iterator.remove

NO-JIRA: update release instructions to reflect change to CDN, and clarify some bits

ARTEMIS-3799 Fix default ping command for Windows

The Windows ping command doesn't fail on destination host unreachable

NO-JIRA Fixing Thread Leakage on test from UDP thread

When running a UDP connection factory you have to either keep it running, or close it so the UDP thread is closed.
this is an issue for the testsuite as we validate for leaked threads. This needs to be fixed on the test.

ARTEMIS-3804: make the version number available to reference in the user manual, add it to the intro page

Updates release instructions to note using the filtered scratch area from artemis-website for other builds

ARTEMIS-3801 fix test

ARTEMIS-3800 Fixing failing test

ARTEMIS-3801 not getting messages on MQTT subscriptions with $

ARTEMIS-3800 allow CLI to delete address with queues

Revert "ARTEMIS-3707 ResourceAdapter Documentation"

This reverts commit 6b784daff1bb3ff178e4b501d6c60bdcf0b456ca.

[ARTEMIS-3799]: Ping command is wrong on Windows.

* Changing the default ping command for Windows.

Jira: https://issues.apache.org/jira/browse/ARTEMIS-3799

This closes #4040

ARTEMIS-3798 Allow binary deserialization of message properties AMQ_MSG_MESSAGE_ID, AMQ_MSG_ORIG_DESTINATION, AMQ_MSG_PRODUCER_ID & AMQ_MSG_REPLY_TO from existing messages.

co-authored: Justin Bertram contributed OpenWireMessageConverterTest changes

ARTEMIS-3707 ResourceAdapter Documentation

ARTEMIS-3707 fix ra.xml

* update xml schemas
* fix ConnectionParameters
* remove TransactionManagerLocatorMethod and "The producer max rate"
* remove The max connections

ARTEMIS-3753 Small Improvement on RouterContext.internalOnly calculation

This closes #4012

ARTEMIS-3753 Prevent sending message to internal queues on mirror

In cluster configuration messages could be routed to internal queues for
further delivering on different broker. We need to check that before
sending to SNF, otherwise message can stuck on target server and will
never receive ACK.

co-author: Clebert Suconic

Discusssions on https://github.com/apache/activemq-artemis/pull/4012 and https://github.com/apache/activemq-artemis/pull/4038

ARTEMIS-3756 Removing test that is now invalid

ARTEMIS-3752: update to Spring 5.3.19 and Spring Boot 2.6.7 which uses it

This closes #4009

[ARTEMIS-3756]: Enabling extra parameters to be passed by the URI.

 * passing the full list of parameters when creating the URI.
 * moving the WARN message to DEBUG when unknown parameters are
   received.

Jira: https://issues.apache.org/jira/browse/ARTEMIS-3756

ARTEMIS-3755 mitigate NPE related to fail-back

ARTEMIS-3771 Avoid not needed lookup for address on OpenWire connections

ARTEMIS-3788 don't send MQTT will message if will flag = false

This closes #4037

ARTEMIS-3789 respect session expiry interval on MQTT disconnect message

ARTEMIS-3791 Openwire failover leaving sessions leaked

ARTEMIS-3765 Fixing testsuite

we cannot really change the attribute's name as it may break user's applications.
I am adding the previous property here.

ARTEMIS-3786 - ensure named properties config from routerConfiguration can be created from string property value

ARTEMIS-3780 - OpenWire, UTF8Buffer to String before setting properties to avoid exception

ARTEMIS-3781 send PUBREC on duplicate PUBLISH for MQTT QoS 2

The MQTT 5 (and 3.1.1) specification states:

    Until it has received the corresponding PUBREL packet, the receiver
MUST acknowledge any subsequent PUBLISH packet with the same Packet
Identifier by sending a PUBREC. It MUST NOT cause duplicate messages to
be delivered to any onward recipients in this case [MQTT-4.3.3-10].

The broker prevents a duplicate message, but it doesn't respond with a
PUBREC. This commit fixes that.

NO-JIRA Fix MQTT trace logging NPE

ARTEMIS-3783: rationalise config for alternate jakarta spec versions to simplify and align their use

ARTEMIS-3782: use existing managed version for maven-shade-plugin uses

This closes #4019

ARTEMIS-3765 Display validated user in Hawtio

This PR adds a "Validated User" column to the Hawtio plugin. The column is
not visible by default. The "Validated User" column is available in the
Session, Consumer, and Producer tabs of the artemis-navigation notebook.
"Validated User" is also available for sort and filtering in those view.
The "Validated Consumer" in the Producer view is always blank. I debugged
as far as I could, but did not find the cause of the issue.

ARTEMIS-3775 don't carry forward MQTT topic aliases to new connection

ARTEMIS-3777 fix MQTT request/response + nolocal

Removing the connection ID property from the actual *message* breaks the
nolocal functionality. Removing the property isn't necessary in the
first place so this commit reomves that code.

NO-JIRA improve MQTT trace logging

This closes #4030

ARTEMIS-3751 Do not override large message size

It is possible to receive a compressed message from the client as regular message. Such a message will already contain correct body size, that takes compression into account.

ARTEMIS-3779 Critial IO Exception logged as Error

ARTEMIS-3778 Just avoiding NPE on Expiry Reaper

Added an extra check on null, just in case
it should not happen, but I preferred adding this clause on the safe side

ARTEMIS-3778 Streamline Expiration Reaping

Instead of holding a thread and an iterator, we should instead keep moving to next references
without holding any threads. Just with callbacks.

ARTEMIS-3776 just my OCD applying aesthetic tweaks to recediveOW.groovy

ARTEMIS-3776 Avoid Integer.MAX_VALUE overflow on openwire clients

Older versions of Openwire clients wil be affected by AMQ-6431.
As a result of the issue if the ID of the message>Integer.MAX_VALUE
a consumer configured with Failover and doing duplicate detection on the client
will not be able to process duplicate detection accordingly and miss messages.

ARTEMIS-3773 remove defunct rate references from web console

This is a follow-up from ARTEMIS-3397. This commit removes the last
traces of the "rate" queue metric from the web console.

ARTEMIS-3774 support user properties on MQTT will message

ARTEMIS-3772: update to mockito 4.4.0

ARTEMIS-3761 Improve page cleanup to remove messages in the middle of the stream as well

Paging only removes files at the beginning of the stream...
Say you have paged files 1 through 1000...
if all the messages are ack, but one message on file 1 is missing an ack, all the 999 subsequent files would not be removed until all the messages on file 1 is ack.
This was working as engineered, but sometimes devs don't have complete control on their app.
With this improvement we will now remove messages in the middle of the stream as well.

There is also some improvement to how browsing and page work with this

ARTEMIS-3769 Fixing queue browsing iterator avoiding NoSuchElement exception from the Iteration

QueueImpl::browserIterator could throw NoSuchElementException and this is fixing the iterator
Found this while testing ARTEMIS-3761

NO-JIRA Making AmqpFlowControlFailTest more accurate

this test was relying on internal details such as number of credits on the link.
The test was flaky and eventually failing or hunging.

ARTEMIS-3768: update to Qpid JMS 1.6.0

ARTEMIS-3758: clean up lots of tests that never run, remove related files + dependencies and some unnecessary deps

ARTEMIS-3762: update docs build, add version overrides to clear security warnings

This closes #4014

ARTEMIS-3760 Upgrade jackson version to 2.13.2

ARTEMIS-3764 wrong CONNACK return code for MQTT5

This bug is causing tests in o.a.a.a.t.i.m.s.c.ConnectTestsWithSecurity
to fail.

This commit fixes the problem by setting the session's version earlier
in the logic handling the CONNECT packet so that the proper CONNACK
return code can be supplied to the remote client in case of
authentication failure.

ensure temp files are off the base class temp folder rule root to get cleaned up post test

ARTEMIS-3757 - allow system and env var substution of properties config, respect order of file loaded properties and add generic enum converter

ARTEMIS-3752 Upgrade Spring Framework to 5.3.18 for examples and tests

ARTEMIS-3752 Upgrade Spring Framework to 5.3.18

ARTEMIS-3754 Upgrade tomcat-servlet-api to 8.5.78

This closes #3997

ARTEMIS-3744 NPE with empty security-manager config in bootstrap.xml

This closes #3996

ARTEMIS-3741 Cache MBeanInfoHelper results

The utility methods in
`org.apache.activemq.artemis.core.management.impl.MBeanInfoHelper` are
executed *a lot* - especially for Jolokia which is used by the web
console. The `MBeanOperationInfo` and `MBeanAttributeInfo` results are
static and reflection is slow therefore they should not be calculated
over and over again. Rather they should be calculated once and cached
for later use.

Caching these results significantly improves performance. Over the
course of 1,000,000 invocations the difference is several orders of
magnitude. This improves usability substantially when dealing with,
for example, tens of thousands of addresses and/or queues.

ARTEMIS-3740 Upgrade Johnzon

This closes #4002

NO-JIRA addUser is also applicable for ActiveMQBasicSerurityManager

This closes #3998

ARTEMIS-3733 - Destination cache size too small for OpenWire clients

NO-JIRA fix duplicate text in security.md doc

Remove the duplicate text in CertificateLoginModule section of the
the security.md doc

ARTEMIS-3749 e2e-tests improvements

Scripts:
- Fix the preapre-docker.sh to exit with 0 instead of 1 on success

On pom files:
- Change e2e-tests variable names to e2e-tests.xxxxxx for clarity on
  e2e-tests variables
- Add e2e-tests.skipImageBuild variable to control if the docker image
  will be build (defaults to not build)
- Add e2e-tests.dockerfile variable to specify the dockerfile to be
  used (defaults to Dockerfile-centos)
- Bump testcontainers version to 1.16.3
- Add artemis distribution dependency since the docker image build
  depends on it

On ContainerService class:
- Fix exposePorts and exporseFolder to use SELinux shared mode
  otherwise the mount fails on machines with SELinux enabled
- Move the logic to use specific user on container from generic start
  method to broker specific method to avoid affect other images
- Update the broker image name to a more generic name (activemq-artemis
  instead of artemis-centos)
- Update the broker image tag to match with the project version in pom
  file

Fix typo

NO-JIRA add version/upgrade info for 2.21.0

NO-JIRA improve Wait.asserEquals error message

[maven-release-plugin] prepare for next development iteration

[maven-release-plugin] prepare release 2.21.0

NO-JIRA updating git clone URI on Release instructions

ARTEMIS-3734 Correcting typo on documentation for CLI Transfer

ARTEMIS-3710 Deprecate queues config element

NO-JIRA fixing MessagesExpiredPagingTest

There is a reference expiring in one hour. It should not leave page mode.
This is about still flowing messages, not leaving page mode.

ARTEMIS-3708 Collapse key transformer into policy

NO-JIRA fix ManifestTest

As the test needs the generated jms jars to be verified I moved it from
unit-tests to smoke-tests.
Updated the test to look for the correct jars as the originally
specified does not exist.
Update the test to assert against Implementation-Version instead of
ActiveMQ-Version in the manifest file as the ActiveMQ-Version property does not exist.

ARTEMIS-3720 Exposing global-max-messages on the CLI

ARTEMIS-3711 support AMQ_SCHEDULED_DELAY for OpenWire clients