Virtual Host Aliases

[edsiper]

Currently Monkey Virtual Host configuration just accepts one host name, it
must accept aliases in order to avoid to have N virtual hosts defined, eg:

Current:

ServerName? www.somesite.com

Expected:

ServerName? somesite.com www.somesite.com calendar.somesite.com

[charlie_brown]

Add support for aliases

[charlie_brown]

My proposal is to add an "Aliases" optional keyword. For example:

ServerName www.somesite.com
Aliases calendar.somesite.com othersite whatever

(patch attached)

[edsiper]

Thanks for the patch and proposal.

After review and re-think how we should handle this feature, the best one is to handle everything in the single variable ServerName?, on this way we avoid to add an extra configuration key and at code level the parsing is more simple.

This implemented feature must avoid to copy strings when a request is parsed.

Also this feature must take care of the host information when is sent to different parts of the core, not just to match the right hostname.

[edsiper]

Fixed in :

 http://git.monkey-project.com/?p=monkey;a=commitdiff;h=7163f64ed4b757d5bdc43e87c4815ea90e5f0d0c

Now the directive Servername, allow multiple aliases in the same line.

[amog2011]

Also this feature must take care of the host information when is sent to different parts of the core, not just to match the right hostname.

air jordan

[edsiper]

Are you reporting a bug or an unexpected behavior ?, please provide more details.