Don't know why, but it fixes the issue described here:
https://community.igniterealtime.org/thread/55868

org.apache.jasper.JasperException: XML parsing error on file /WEB-INF/web.xml

REST API plugin: make group affiliations per room visible

This patch shows which groups have one of the affiliations
admin|member|owner|outcast.
In case the new query parameter expandGroups is set to true, the group
member's jids are added to the explicit jids.

These are actually not required by Java compiler and therefore have been removed with dfb69a76, but qdox needs them.

OF-938: Apply packet namespace via string insert

This patch fixes a problem introduced by an earlier patch whereby a
change to the Packet namespace inadvertently cascaded to child XML
elements.

OF-966 <failure> tag not closed for TLS Negotitation Failure

Use StandardCharsets.UTF_8 with Reader/Writer classes.

This is a followup for fcf665a9.

Use real Java enums for presence subscription states.

The homemade IntEnum and Enum classes feel totally weird and confusing, when having java.lang.Enum around.

This commit removes these classes.

The static constants in the RosterItem class now feel superfluous, but are kept in there, because they are referenced by a lot of other classes and JSP pages.

Use Collections.singleton() where ArrayList has only one element.

Fix PluginClassLoader load resource path issue  

Use try-with-resources statement to properly close resources.

Replace deprecated org.jivesoftware.util.Log with org.slf4j.Logger

- Not all resources were closed in finally block (see Launcher, PluginManager, UpdateManager, FaviconServlet, ...)
- Get rid of verbose finally blocks and empty catch blocks, which makes the code more readable.
- Generally, this is the more modern approach to deal with resources.

	When load plugin resource files old commit will load all resource with / path

	Like /images/WEB-INF/web.xml

	old commit is /web.xml

	should be /images/WEB-INF/web.xml

Deprecate ConcurrentHashSet in favor of Collections.newSetFromMap().

ConcurrentHashSet seems to be a relict from earlier Java times.

Since Java 1.6 the preferred method to create a "ConcurrentHashSet" is to use Collections.newSetFromMap(new ConcurrentHashMap<?, ?>())

This has performance benefits. On my machine it's nearly twice as fast.

Add PluginClassLoader plugin loader isolation mechanism

fix getAuditedStartDate resetting startDate to null

Replace Arrays.asList() (with 1 arg) with Collections.singletonList().

Replace StringBuffer with StringBuilder (for performance).

Java Generics: Fix various "unchecked" compiler warnings.

Use Collections.unmodifiableCollection() instead of obsolete Immutabl…

Use integrated AWT SystemTray and TrayIcon classes instead of JDIC.

Make code more readable by using StandardCharsets.UTF_8.

- Use String.getBytes(Charset) instead of String.getBytes(String)
- Use new String(byte[], Charset) instead of new String(byte[], String)
-> Get rid of UnsupportedEncodingException.
-> Less try-catch blocks.

Prefer StringBuilder.append(char) over append(String).

JDIC is obsolete starting with JDK 1.6:
"Most of the features provided by JDIC were incorporated into the JDK starting with version 1.6. As a result, the development of JDIC has come to an end."