File 0210-Include-location-metadata-in-the-maybe_cli... of Package erlang

Overview Repositories Revisions Requests Users Attributes Meta

File 0210-Include-location-metadata-in-the-maybe_client_warn_n.patch of Package erlang (Revision cb0c7ad891d5307a2eb53842d7d69a0f)

Currently displaying revision cb0c7ad891d5307a2eb53842d7d69a0f , Show latest

From 7f3448dbbe6856da843fe7b21c1006a736d2cbe4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Diego=20Elio=20Petten=C3=B2?= <flameeyes@fb.com>
Date: Tue, 15 Mar 2022 04:58:16 -0700
Subject: [PATCH] Include location metadata in the maybe_client_warn_no_verify
 log.

This is the only ssl_logger:log/4 call that passed an empty metadata
rather than providing at least the location.
---
 lib/ssl/src/ssl.erl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/ssl/src/ssl.erl b/lib/ssl/src/ssl.erl
index d87f447a26..a0217198ae 100644
--- a/lib/ssl/src/ssl.erl
+++ b/lib/ssl/src/ssl.erl
@@ -26,6 +26,7 @@
 -module(ssl).
 
 -include_lib("public_key/include/public_key.hrl").
+-include_lib("kernel/include/logger.hrl").
 
 -include("ssl_internal.hrl").
 -include("ssl_api.hrl").
@@ -2865,7 +2866,7 @@ maybe_client_warn_no_verify(#{verify := verify_none,
                              warn_verify_none := true,
                              log_level := LogLevel}, client) ->
             ssl_logger:log(warning, LogLevel, #{description => "Authenticity is not established by certificate path validation",
-                                                reason => "Option {verify, verify_peer} and cacertfile/cacerts is missing"}, #{});
+                                                reason => "Option {verify, verify_peer} and cacertfile/cacerts is missing"}, ?LOCATION);
 maybe_client_warn_no_verify(_,_) ->
     %% Warning not needed. Note client certificate validation is optional in TLS
     ok.
-- 
2.34.1

Places

File 0210-Include-location-metadata-in-the-maybe_client_warn_n.patch of Package erlang (Revision cb0c7ad891d5307a2eb53842d7d69a0f)

Places