Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple network interfaces for a vhost? #31

Open
jamesyoungman opened this issue Apr 19, 2019 · 4 comments
Open

Multiple network interfaces for a vhost? #31

jamesyoungman opened this issue Apr 19, 2019 · 4 comments

Comments

@jamesyoungman
Copy link

Is there any way to support multiple IP adresses for a VM?

I would like to do this because I need to have more than one Apache virtual host on a VM and allow access to port 80 on both addresses. I need to do that because one of the vhosts needs to be configured with some insecure settings ("HttpProtocolOptions Unsafe") which I wouldn't want to apply to the default vhost.

Right now I don't think I can have two IP addresses because the hooks script assumes that there is exactly one possible value of private_ip per VM, I think.

I can't make one of the Apache vhosts a CNAME (and thus need only one IP address) because this is against best practice for Apache httpd (see https://httpd.apache.org/docs/2.4/dns-caveats.html).

@Doralitze
Copy link
Contributor

I also do require multiple IP Addresses per VM but with different port forwards. So I'll hack something and make a PR about it.
As my understanding goes this should be as simple as allowing the hook for a particular VM to handle multiple sections as different addresses are handled by different rule sets as far as iptables goes.

There are multiple PR which aren't addressed but others are. Are there any rules about that?

@saschpe
Copy link
Owner

saschpe commented Jan 13, 2021

Well, I stopped using this tool a while ago so I can't really test bigger PRs. There are quite a few other people who do however and sometimes they chime in. I do tend to merge those that are straightforward to review though.

@Doralitze
Copy link
Contributor

Thanks for the quick answer. I'll write those changes for myself then and maybe they'll end up being upstreamed. We'll see.

@Doralitze
Copy link
Contributor

I've created a pull request (#34) addressing this issue. I hope it might get merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants