8000 Hosts parser change by ibauersachs · Pull Request #371 · dnsjava/dnsjava · GitHub
[go: up one dir, main page]
Skip to content

Hosts parser change #371

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Hosts parser change #371

wants to merge 2 commits into from

Conversation

ibauersachs
Copy link
Member

No description provided.

@ibauersachs
Copy link
Member Author

@ivanr-rs can you please verify if these changes work for you?
@tzaeschke is this enough log reduction?

@ibauersachs ibauersachs changed the title Hsots parser change Hosts parser change Jan 19, 2025
@tzaeschke
Copy link

Thanks for looking into this!
I tried it and the original warning has disappeared.
However, I now see a different warning:
[main] WARN org.xbill.DNS.hosts.HostsFileParser - Failed to parse entire hosts file /etc/hosts, address failures=3, name failures=0
It is an improvement in cases where there are multiple of our entries in the /etc/hosts file, otherwise it is just a different log message.
I would still prefer no warning at all, just like the JDK's or OS's own DNS resolver, if possible.

@ibauersachs
Copy link
Member Author

@tzaeschke I know that the log is still there, that was intentional, and I'm still not inclined to remove it. What I was wondering for few days now: why are you writing to /etc/hosts if only your software can parse those records?

@tzaeschke
Copy link
tzaeschke commented Jan 22, 2025 •
edited
Loading

@ibauersachs The central part of our project is a new routing protocol (essentially replacing BGP), see https://www.scion.org/. I am working on a Java client, and there are clients in other languages (Golang, Rust, ...). In the end, many applications on a machine, including possibly the kernel/OS itself, should use the new protocol. We support using a separate hosts file in /etc/scion/hosts, but it would be good to keep using the /etc/hosts, otherwise it may become more or less useless for any addresses outside the local network (local autonomous system), and we end up with two separate configuration files to specify host addresses.
To many, replacing BGP sounds like an impossible undertaking, but several ISPs (including British Telecom and all major Swiss ISPs) now support SCION. Also, the complete Swiss banking network has migrated to SCION. The reference implementations of all components are free open source owned by a non-profit organization.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ibauersachs @tzaeschke
0