Revert "Revert "[WFLY-2047] org.jboss.as.connector module needs sun.jdk module dependency""

This fix is harmless, even if not strictly "correct", and it solves real-world problems.  There was no good reason to revert this commit in the first place.

This reverts commit 9a3b000767c018a681a4858a84aae20a0b8c7310.
The original revert reverted 88756ddb1061660cb.
[WFLY-2672] Add TRACE logging for LDAP group loading.
Upgrade to HAL 2.1.1.Final
WFLY-2829 Optimistic locking w/write skew check requires versioning to be enabled
[WFLY-1597] / [WFLY-2208] Ensure that any HTTP to HTTPS redirect occurs BEFORE upgrade.

That is if the HTTP interface is configured with a secure port we do not support any unencrypted management access.
Don't modify the top level deployment's module spec from a sub deployment
[WFLY-1597] / [WFLY-2208] Handle a redirect from HTTP to HTTPS and convert to the correct address format for https-remoting.
WFLY-2819 Upgrade jsf-impl to 2.2.5-jbossorg-3
[WFLY-2759] Upgrade to HornetQ 2.4.1.Final

* upgrade hornetq version to 2.4.1.Final
* add new initial-connect-attempts attribute for cluster-connection and
  bridge resources [WFLY-2758]

JIRA: https://issues.jboss.org/browse/WFLY-2759
JIRA: https://issues.jboss.org/browse/WFLY-2758
[WFLY-2730] HTTP upgrade multiplexing

When the HTTP upgrade handshake is performed, check whether
the http-upgrade-endpoint is defined and accepts the handshake
only if its value matches the acceptorName.
This http-upgrade-endpoint header is used by a http-connector to
specify which http-acceptor must be used in case there are multiple
http-acceptors registered on the server.

If the http-upgrade-endpoint header is not present, the 1st http-acceptor
encountered will be used. This is not deterministic and is suitable only
if there is a single http-acceptor.

* remove native messaging port for messaging subsystem.xml configuration
and use http-connector for JMS RemoteConnectionFactory
* add a http-connector-throughput to replace the corresponding
  netty-throughput

JIRA: https://issues.jboss.org/browse/WFLY-2730
[WFLY-2834] The class MSCTestRunner was never completed or used.
[WFLY-2584] Throw an IllegalArgumentException is a user attempts to switch to a role that does not exist.

Note: This is only thrown for users that would have the ability to switch roles if the role actually existed - other users it still fails silently to stop them from using this to discover which roles actually exist.
Merge pull request #5809 from stuartwdouglas/WFLY-2496

Don't modify the top level deployment's module spec from a sub deploymen...
Merge pull request #5805 from darranl/WFLY-2672

[WFLY-2672] Add TRACE logging for LDAP group loading.
Merge pull request #5817 from darranl/WFLY-2208

[WFLY-1597] / [WFLY-2208] Ensure http to https redirect is before HTTP upgrade and update CLI to honour the redirect.
Merge pull request #5818 from darranl/WFLY-2834

[WFLY-2834] The class MSCTestRunner was never completed or used.
Merge pull request #5821 from darranl/WFLY-2584

[WFLY-2584] Throw an IllegalArgumentException is a user attempts to swit...
Merge pull request #5785 from dmlloyd/connector-sun.jdk

Revert "Revert "[WFLY-2047] org.jboss.as.connector module needs sun.jdk module dependency""
Merge pull request #5806 from hpehl/master

WFLY-2827: Upgrade to HAL 2.1.1.Final
Merge pull request #5808 from pferraro/master

WFLY-2829 Optimistic locking w/write skew check requires versioning to be enabled