Skip to content

Add x509-async module#435

Merged
hannesm merged 2 commits into
mirleft:mainfrom
torinnd:x509-async
Jul 15, 2021
Merged

Add x509-async module#435
hannesm merged 2 commits into
mirleft:mainfrom
torinnd:x509-async

Conversation

@torinnd
Copy link
Copy Markdown
Contributor

@torinnd torinnd commented Jul 1, 2021

This is a subset of the changes proposed in mirleft/ocaml-x509#150. At a high level:

  • Cstruct.t is replaced with contents:string for various decode functions
  • ('a, [ `Msg of string ]) result is replaced with 'a Or_error.t
  • of_pem_file and of_pem_dir functions are provided where it's useful to go from a Filename.t -> t or t list
  • [Authenticator] gets a [Param] submodule to facilitate constructing an Authenticator.t where I/O is required

This Pull Request doesn't include the sexplib or to_string opinions proposed in mirleft/ocaml-x509#150 as they're relatively easy to inline.

The example server test_server.ml is updated to use the new call sites.

@hannesm hannesm merged commit 7ea229b into mirleft:main Jul 15, 2021
@hannesm
Copy link
Copy Markdown
Member

hannesm commented Jul 15, 2021

thanks.

hannesm added a commit to hannesm/opam-repository that referenced this pull request Aug 2, 2021
CHANGES:

* Breaking: peer_name (in config and epoch data, also own_name) is now a
  [`host] Domain_name.t instead of a string. (mirleft/ocaml-tls#434 mirleft/ocaml-tls#438 @torinnd @hannesm)
* Add a X509_async module (mirleft/ocaml-tls#435 @torinnd)
* Client and server constructor log messages are on the debug level (mirleft/ocaml-tls#436
  reported by @talex5, fix by @hannesm)
* Adapt to cstruct 6.0.0 API (Cstruct.len is deprecated) mirleft/ocaml-tls#439 @hannesm
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants