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

Dns_client_mirage: allow hostname in authenticator #319

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

hannesm
Copy link
Member

@hannesm hannesm commented Oct 5, 2022

//cc @reynir

@hannesm hannesm changed the title Dns_client_mirage: allow hostname in authenticator, use ":" as separa… Dns_client_mirage: allow hostname in authenticator Oct 5, 2022
Copy link
Member

@reynir reynir left a comment

Choose a reason for hiding this comment

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

Good catch!

@hannesm hannesm merged commit b7be3de into mirage:main Oct 5, 2022
@hannesm hannesm deleted the allow-hostname branch October 5, 2022 20:07
hannesm added a commit to hannesm/opam-repository that referenced this pull request Oct 24, 2022
…er, dns-mirage, dns-client, dns-cli and dns-certify (6.4.0)

CHANGES:

* dns-client: demote log level of response to debug (mirage/ocaml-dns#317 @hannesm)
* dns-client: use DNS-over-TLS for uncensoreddns.org only (mirage/ocaml-dns#320 @hannesm)
* API: dns-client: connect returns the protocol (UDP/TCP), allowing mixed UDP
  and TCP namerservers being used (mirage/ocaml-dns#322 @hannesm)
* dns-client-mirage: allow hostname in authenticator, improve error message and
  documentation (mirage/ocaml-dns#319 mirage/ocaml-dns#322 @hannesm)
* dns-client-mirage: support UDP nameservers as "udp:<IP>" in
  nameserver_of_string (mirage/ocaml-dns#322 @reynir @hannesm)
* API: dns-client, dns-stub, dns-resolver: ?size is now ?cache_size (mirage/ocaml-dns#322
  @hannesm, suggested by @reynir)
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