wildfly/wildfly — pull request #6396 by quijote (2014-06-17 15:43:37+00:00)
Title: ServletContext.getVirtualServerName always returns "localhost"

I experimented with the new getVirtualServerName method introduced for interface ServletContext in servlet spec 3.1. The JavaDoc states:
> Returns the primary name of the virtual host on which this context is deployed. The name may or may not be a valid host name.

However Wildfly always seems to return "localhost" instead of the virtual host name specified in XML configuration files. My host config in standalone.xml looks like this: ``. I expected that getVirtualServerName returns "default-host" in this case.

getVirtualServerName is implemented like this: `return deployment.getDeploymentInfo().getHostName();`. However DeploymentInfo.setHostName is never called during startup.

If I did not misunderstand what getVirtualServerName is supposed to deliver, something similar to my proposed patch should fix the problem. Sorry, I only looked at Wildfly source code for the first time today in a debugger and was unable to verify if my proposed patch compiles :( Hope a more experienced developer can have a quick look and verify if this fix is valid.