[gnutls-devel] GnuTLS | WIP: RFC7250 Raw public keys (!650)
Development of GNU's TLS library
gnutls-devel at lists.gnutls.org
Tue Oct 30 11:42:56 CET 2018
Tom commented on a discussion on lib/includes/gnutls/gnutls.h.in:
> unsigned idx,
> gnutls_datum_t * response);
> +/* RAW public key functions (RFC7250) */
> +#ifdef ENABLE_RAWPK
> +int gnutls_certificate_set_rawpk_keypair(gnutls_certificate_credentials_t cred,
> + const gnutls_pubkey_t subjectPublicKeyInfo,
> + const gnutls_privkey_t key);
> +int gnutls_certificate_set_rawpk_keypair_raw(gnutls_certificate_credentials_t cred,
I mean how about following the naming convention from `pcert.c`. There you use the `_raw` suffix when plain `datum_t` types are being fed into the functions. E.g. `gnutls_pcert_import_x509_raw` and `gnutls_pcert_import_x509`. And then create an alias for the `...set_x509_key_mem` functions.
Also we can think about renaming these key set functions to using "keypair" in the name instead of "key" to make clear its about a key pair. Of course we need to stay API compatible and need to create aliases for the old ones.
Just a wild idea to improve code readability.
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/merge_requests/650#note_113100454
You're receiving this email because of your account on gitlab.com.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Gnutls-devel