Skip to content

crypto: simplify Certificate class bindings

Rodrigo Muino Tomonari requested to merge github/fork/estliberitas/crypto-certificate-static into master

Description of change

Replace Certificate C++ class with simple functions. Update crypto.Certificate methods accordingly.

Original talk was in comments of #5230.

I removed Certificate C++ class from crypto bindings and replaced its methods with simple functions. I also prefixed function names with cert in order to separate these functions from others like randomBytes, etc. (to be discussed).

Pull Request check-list

  • Does make -j8 test (UNIX) or vcbuild test nosign (Windows) pass with this change (including linting)?
  • Is the commit message formatted according to CONTRIBUTING.md?
  • If this change fixes a bug (or a performance problem), is a regression test (or a benchmark) included?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Affected core subsystem(s)

  • crypto

Merge request reports