-
Notifications
You must be signed in to change notification settings - Fork 51
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
Check 'hosts' file records, during handling DNS requests #401
Check 'hosts' file records, during handling DNS requests #401
Conversation
0d206fc
to
7f9dc16
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some questions/comments, but overall looks good!
pkg/services/dns/hosts_file.go
Outdated
|
||
func (h *hosts) updateHostsFile() error { | ||
h.hostsReadLock.RLock() | ||
defer h.hostsReadLock.RUnlock() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method is updating data structures (h
) shared between multiple go routines, this should be taking a write lock (WLock()
)
You don't need to do it before the readHostsFile
call though, as it's not changing any shared state, this could be moved just before h.hostsFile = newHosts
This implementation use 'libhosty' lib to read and search for hosts file records. It reads 'hosts' file first and adds file watcher to track changes in 'hosts' file Signed-off-by: Yevhen Vydolob <[email protected]>
7f9dc16
to
3ea66d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One pending comment, otherwise
/lgtm
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cfergeau, evidolob The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
de690ca
into
containers:main
This implementation use 'libhosty' lib to read and search for hosts file records. It reads 'hosts' file first and adds file watcher to track changes in 'hosts' file