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

enable listening on IPv6 #99

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

toddjcrane
Copy link

small modification to allow listening on IPv6 addresses

@toddjcrane
Copy link
Author

I still need to write tests for this

@jonashaag
Copy link
Owner

Hi thanks for the patch. Minor things:

  • how do I listen on all IPv4 interfaces? With your code it's possible to only listen on all IPv6 interfaces if I understand it correctly
  • there's repeated code there that can easily be put into a function or conditional statement

Tests aren't really necessary, just look at all the other test cases, none if them are actual tears, more like examples...

@toddjcrane
Copy link
Author

To listen on all IPv4, listen on 0.0.0.0
I thought about it and decided to set the default to listen on IPv6 as listening on all IPv6 will also listen on all IPv4. Also, there was a bit of a (not so) hidden agenda to further IPv6 enable the internet.

I'll make the refactoring changes to eliminate repetitive code and I'll have it up here shortly

@holms
Copy link

holms commented Jan 16, 2017

Can this be confirmed please?

@jonashaag
Copy link
Owner

The patch needs some minor refactoring before it can be landed.

@tambry tambry mentioned this pull request Mar 17, 2018
@danigosa danigosa mentioned this pull request Jun 15, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants