-
-
Notifications
You must be signed in to change notification settings - Fork 110
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
Can not connect to unix socket file #19
Labels
enhancement
New feature or request
Comments
@herb123456 Unix sockets were not a priority for us, so thanks a lot for your pull request. Looking at the code we probably need to find a way to integrate #21 Thoughts? |
that's ok. I will send another pull request a few days later. |
estahn
added a commit
that referenced
this issue
Mar 17, 2018
…herb123456 for the initial implementation) #19 #22
estahn
added a commit
that referenced
this issue
Mar 18, 2018
* Implement unix socket support and make status path configurable (Thx @herb123456 for the initial implementation) #19 #22 * Follow linting guidelines * Follow linting guidelines * Move `env` to where it's being used
@herb123456 This issue has been fixed with v0.5.0. Thanks again for your pull request. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
then test using curl output
just because uri.Hostname() and uri.Port() is empty when address is unix socket file path at phpfpm/phpfpm.go Line160
I will send a pull request for this issue later.
The text was updated successfully, but these errors were encountered: