-
-
Notifications
You must be signed in to change notification settings - Fork 66
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
provide a workaround for ruby/resolv/issues/23 #834
provide a workaround for ruby/resolv/issues/23 #834
Conversation
If I get this correctly, |
Please see the details here #369 The issue I have is that within my network environment the docker host itself cannot resolve any network name, therefore some nameservers are configured, but unfortunately due to the bug (ruby/resolv/issues/23) I need to "patch" the Currently I add an extra env with this value |
Since you are building custom images, have you tried adding that to your Dockerfile with a custom
|
That's what I'm doing right now, but to be honest I would really prefer to use your image out-of-the-box, without any modifications from my site. As this has the downside that I frequently have to update and maintain my images, if they do not work with an updated extension. That's why I really would like to ask you to add this minor and non breaking change again here - even if this is against your code cleanness (which I completely understand). 🙏 I'll continue to get the real fix PR merged @ruby/resolv - but unfortunately there I did not get any response up to now. |
That's why I really would like to ask you to add this minor and non breaking change again here - even if this is against your code cleanness (which I completely understand). 🙏 I'll continue to get the real fix PR merged @ruby/resolv - but unfortunately there I did not get any response up to now. |
c1e74f7
to
7f7224b
Compare
Hi @mburumaxwell,
this PR revives the changes that PR #369 brought with it and unfortunately were lost some time ago.
Can you please add this once again, until Ruby
Resolv
bug gets merged.Thanks a lot.