[gnutls-devel] GnuTLS | priority: support allowlisting in configuration file (!1427)

[Read-only notification of GnuTLS library development activities]

Alexander Sosedkin commented:


Not a complete review by any means, more like field notes.
I got to play with it, and with a rebase on top of 4af2969d50,

```
+++ a/lib/algorithms/ecc.c
+++ b/lib/algorithms/ecc.c
@@ -295,7 +295,7 @@ const gnutls_ecc_curve_t *gnutls_ecc_curve_list(void)
 unsigned _gnutls_ecc_curve_is_supported(gnutls_ecc_curve_t curve)
 {
        GNUTLS_ECC_CURVE_LOOP(
-               if (p->id == curve && p->supported && _gnutls_pk_curve_exists(p->id))
+               if (p->id == curve && (p->supported || p->supported_revertible) && _gnutls_pk_curve_exists(p->id))
                        return 1;
        );
        return 0;

```

and `prio` containing
```
[global]
override-mode = allowlist

[overrides]
secure-hash = SHA256
tls-enabled-mac = AEAD
secure-sig = RSA-SHA256
secure-sig-for-cert = RSA-SHA256
tls-enabled-cipher = AES-256-GCM
tls-enabled-kx = RSA
enabled-version = TLS1.2
min-verification-profile = medium


[priorities]
SYSTEM=NONE
```

I was able to `GNUTLS_SYSTEM_PRIORITY_FILE=prio ./src/gnutls-cli -d999 google.com --priority @SYSTEM`.

Not sure why `secure-sig-for-cert` and `--priority @SYSTEM` were needed.

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/-/merge_requests/1427#note_602107176
You're receiving this email because of your account on gitlab.com.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.gnupg.org/pipermail/gnutls-devel/attachments/20210615/c2874bdf/attachment.html>