[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

ARTEMIS-3720 Improvements on Paging MaxMessages usage

I "used" the broker a little bit around max-messages and found a few minor issues.

This closes #3994

[ARTEMIS-3732]: ServerLocator disableFinalizeCheck method has been removed without being deprecated first.

* Adding back empty default method for retro compatibility.

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

This closes #3980

ARTEMIS-3716 move e2e tests to e2e-tests module

Move all tests which are related to end-to-end testing from smoke-tests
module to a new module named e2e-tests.

These e2e tests are those which are dependent of ContainerService
class. ContainerService class uses artemis inside a container by using
the testcontainers library and for that reason these tests are usually
a quite slow and tecnically they are not a smoke test.

The new e2e-tests module is part of tests module but it is not enabled
by default and to get executed it requires the e2e-tests profile
specification on maven command.

NO-JIRA Simplifying some formatting in logging

ARTEMIS-3709 Add group-rebalance-pause-dispatch attribute to queueType

ARTEMIS-3676 ignore NoRouteToHostException

This kind of exception can be common in cloud environments and should be
ignored for logging just like ConnectException.

ARTEMIS-3729 Fix JMS CORE client commit after async sends

ARTEMIS-3720 Documentation about max-messages

ARTEMIS-3720 Max Number Of Messages on a deciding policy for paging (global and address-settings)

ARTEMIS-3702 auth failures don't adhere to MQTT spec

The commit includes the following changes:
 - Don't drop the connection on subscribe or publish authorization
failures for 3.1 clients.
 - Don't drop the connection on subscribe authorization failures for
3.1.1 clients.
 - Add configuration parameter to control behavior on publish
authorization failures for 3.1.1 clients (either disconnect or not).

ARTEMIS-3715 Support testClientID for artemis-maven-plugin

ARTEMIS-3724: Update Jetty to 9.4.45.v20220203

ARTEMIS-3723: update to Netty 4.1.75 (and netty-tcnative 2.0.50)

ARTEMIS-3721 AMQP Mirrored Large Message file not removed

ARTEMIS-3719 DLA and expiry incorrect w/temp-queue-namespace

When using a temporary queue with a `temporary-queue-namespace` the
`AddressSettings` lookup wasn't correct. This commit fixes that and
refactors `QueueImpl` a bit so that it holds a copy of its
`AddressSettings` rather than looking them up all the time. If any
relevant `AddressSettings` changes the
`HierarchicalRepositoryChangeListener` implementation will still
refresh the `QueueImpl` appropriately.

The `QueueControlImpl` was likewise changed to get the dead-letter
address and expiry address directly from the `QueueImpl` rather than
looking them up in the `AddressSettings` repository.

I modified some code that came from ARTEMIS-734, but I ran the test that
was associated with that Jira (i.e.
`o.a.a.a.t.i.c.d.ExpireWhileLoadBalanceTest`) and it passed so I think
that should be fine. There actually was no test included with the
original commit. One was added later so it's hard to say for sure it
exactly captures the original issue.

ARTEMIS-3591: stop paging checkMemory task executing twice

This closes #3977

ARTEMIS-3714 change Docker group and user IDs to 1001

Changed group and user ids to 1001 to allow Artemis user to be the own
of /var/lib/artemis-instance and /opt/activemq-artemis. Otherwise the
owner will be root (1000).

This was already done in Dockerfile-centos file.

This closes #3981

ARTEMIS-3712 refresh doc generation tool-chain