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

Reinit the dns lib before each call #1891

Merged
merged 4 commits into from
Apr 9, 2020
Merged

Reinit the dns lib before each call #1891

merged 4 commits into from
Apr 9, 2020

Conversation

karknu
Copy link
Contributor

@karknu karknu commented Apr 2, 2020

The dns library only reads /etc/resolv.conf once which means that inorder to
cope with changing networks conditions we need to reinit the library
before each use.

Workaround for the bug reported in IntersectMBO/cardano-node#731

karknu added 3 commits April 6, 2020 11:19
The dns library only reads /etc/resolv.conf once which means that inorder to
cope with changing networks conditions we need to reinit the library
before each use.
@karknu karknu requested a review from coot April 6, 2020 09:21
Copy link
Contributor

@coot coot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@karknu
Copy link
Contributor Author

karknu commented Apr 9, 2020

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Apr 9, 2020

@iohk-bors iohk-bors bot merged commit 3d89fa4 into master Apr 9, 2020
@iohk-bors iohk-bors bot deleted the karknu/dns branch April 9, 2020 19:42
coot added a commit that referenced this pull request May 16, 2022
1891: Reinit the dns lib before each call r=karknu a=karknu

The dns library only reads /etc/resolv.conf once which means that inorder to
cope with changing networks conditions we need to reinit the library
before each use.

Workaround for the bug reported in IntersectMBO/cardano-node#731 

Co-authored-by: Karl Knutsson <[email protected]>
Co-authored-by: Marcin Szamotulski <[email protected]>
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