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

Remove ipv6 support and add logging #8

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

Conversation

e1z0
Copy link

@e1z0 e1z0 commented Jul 16, 2021

Remove ipv6 support that causes the error "Address family not supported by protocol" and add logging support to services

…ed by protocol" and add logging support to services
@jackyzy823
Copy link
Owner

Thanks for your PR and sorry for late response because i don't have too much time on this project.

It's pretty weird that i have not encounter the ipv6 "Address family not supported by protocol" issue of nginx under docker . I'll remove ipv6 listen later.

In my mind, those services without logging is a kind of infra so i decide not to log it.

Anyway since this is just a template project, you can modify anything as you like.

Besides json-file is not recommended by docker. Docker recommends local.

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.

2 participants