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

Also add bound address property when emitting the carapace::port event. #9

Closed
wants to merge 1 commit into from

Conversation

stolsma
Copy link
Contributor

@stolsma stolsma commented Sep 23, 2011

Also adding the bound address when emitting the carapace::port event will give the possibility to use a different proxy for ipv6 and for ipv4 ports.

It also makes it possible to selectively (i.e. only for that ipv4/ipv6 address) open up the requested port in a firewall.

@stolsma
Copy link
Contributor Author

stolsma commented Sep 27, 2011

Can somebody push this to master ?? I need something like this for a firewall hook.... :-(

@Marak
Copy link
Contributor

Marak commented Sep 27, 2011

We are a bit backed up with jsconf this week, but I will make sure someone looks at this.

Thank you for all your pull requests, they have been very helpful!

:-)

@stolsma
Copy link
Contributor Author

stolsma commented Sep 27, 2011

Sorry, should have known that, looks like its time to go to sleep before I start yelling at the cat... ;-) Are you also going to Amsterdam like indexzero?

@Marak
Copy link
Contributor

Marak commented Sep 27, 2011

I will only be in Berlin at jsconf, no Amsterdam for me.

@indexzero
Copy link
Member

Cherry-picked in a72c9db. Will be published tonight

@indexzero indexzero closed this Oct 9, 2011
@stolsma
Copy link
Contributor Author

stolsma commented Oct 10, 2011

Thanks!! Will try to add something like a firewall plugin (or example) to haibu later on when I have some more time...

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

Successfully merging this pull request may close these issues.

3 participants