diff options
| author | Dr. David von Oheimb <David.von.Oheimb@siemens.com> | 2023-06-11 17:36:55 +0200 |
|---|---|---|
| committer | Tomas Mraz <tomas@openssl.org> | 2023-10-10 20:36:05 +0200 |
| commit | 4a9299ac5090dc7997bd1f2cbc56e5e11c6277ff (patch) | |
| tree | 4be1af79345dccfbb28a49f7ff07657cb98eb1d0 /apps/cmp.c | |
| parent | 8d120aef951d7bb7deac0b8b559f8003f5ea6384 (diff) | |
apps/cmp.c: -tls_used may be implied by -server https:...; improve related checks and doc
Reviewed-by: Todd Short <todd.short@me.com>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/21176)
Diffstat (limited to 'apps/cmp.c')
| -rw-r--r-- | apps/cmp.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/apps/cmp.c b/apps/cmp.c index e38f0010a0..911d94c198 100644 --- a/apps/cmp.c +++ b/apps/cmp.c @@ -1933,10 +1933,6 @@ static int setup_client_ctx(OSSL_CMP_CTX *ctx, ENGINE *engine) CMP_warn("ignoring -proxy option since -server is not given"); if (opt_no_proxy != NULL) CMP_warn("ignoring -no_proxy option since -server is not given"); - if (opt_tls_used) { - CMP_warn("ignoring -tls_used option since -server is not given"); - opt_tls_used = 0; - } goto set_path; } if (!OSSL_HTTP_parse_url(opt_server, &use_ssl, NULL /* user */, @@ -1946,8 +1942,8 @@ static int setup_client_ctx(OSSL_CMP_CTX *ctx, ENGINE *engine) goto err; } if (use_ssl && !opt_tls_used) { - CMP_err("missing -tls_used option since -server URL indicates HTTPS"); - goto err; + CMP_warn("assuming -tls_used since -server URL indicates HTTPS"); + opt_tls_used = 1; } BIO_snprintf(server_port, sizeof(server_port), "%s", port); |