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

Added support for /etc/hosts #10

Closed
attozk opened this issue Oct 15, 2014 · 10 comments · Fixed by #75
Closed

Added support for /etc/hosts #10

attozk opened this issue Oct 15, 2014 · 10 comments · Fixed by #75

Comments

@attozk
Copy link

attozk commented Oct 15, 2014

reactphp/reactphp#225 should be ported over

@cboden
Copy link
Member

cboden commented May 10, 2015

+1

I think we're ok to read the filesystem even if it blocks in the initialization phase, as long as it's not once the loop has entered it's run phase.

@attozk
Copy link
Author

attozk commented May 10, 2015

amen

@clue
Copy link
Member

clue commented May 31, 2015

Big +1 for getting this in, both for convenience ("localhost" etc.) and for Docker containers!

I've started looking into this a few weeks ago, but have postponed this for now because the DNS component is currently undergoing some major architectural refactorings.

@fr34kyn01535
Copy link

Yes please!

@nazar-pc
Copy link

nazar-pc commented Jul 8, 2015

Any updates here?

@inwaar
Copy link

inwaar commented Aug 7, 2015

+1 for getting support for /etc/hosts

@ezzatron
Copy link

ezzatron commented Jan 8, 2016

+1

@matthiasnoback
Copy link

Hi everyone, just wanted to say: this feature is very much needed. It's quite counter-intuitive the way it works now, and it has cost me several hours to find out what the problem was.

Just in case someone is using Docker containers and wants to connect to them using ReactPHP DNS: don't use the public 8.8.8.8 DNS server, but the embedded Docker DNS server, which is as far as I can tell always available at 127.0.0.11.

@WyriHaximus
Copy link
Member

@matthiasnoback I've kicked off a PR (#52) to make that happen.

@matthiasnoback
Copy link

@WyriHaximus Excellent!

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

Successfully merging a pull request may close this issue.

9 participants