Merge pull request #10514 from thalesmg/unhide-ocsp-config-r50

chore: un-hide ocsp stapling config
This commit is contained in:
Thales Macedo Garitezi 2023-04-25 10:45:07 -03:00 committed by GitHub
commit 66c302064e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 4 deletions

View File

@ -2323,8 +2323,6 @@ server_ssl_opts_schema(Defaults, IsRanchListener) ->
ref("ocsp"),
#{
required => false,
%% TODO: remove after e5.0.2
importance => ?IMPORTANCE_HIDDEN,
validator => fun ocsp_inner_validator/1
}
)},
@ -2333,6 +2331,7 @@ server_ssl_opts_schema(Defaults, IsRanchListener) ->
boolean(),
#{
default => false,
importance => ?IMPORTANCE_MEDIUM,
desc => ?DESC("server_ssl_opts_schema_enable_crl_check")
}
)}

View File

@ -677,8 +677,12 @@ do_t_update_listener(Config) ->
%% no ocsp at first
ListenerId = "ssl:default",
{ok, {{_, 200, _}, _, ListenerData0}} = get_listener_via_api(ListenerId),
?assertEqual(
undefined,
?assertMatch(
#{
<<"enable_ocsp_stapling">> := false,
<<"refresh_http_timeout">> := _,
<<"refresh_interval">> := _
},
emqx_utils_maps:deep_get([<<"ssl_options">>, <<"ocsp">>], ListenerData0, undefined)
),
assert_no_http_get(),