spacewander commented on a change in pull request #3053:
URL: https://github.com/apache/apisix/pull/3053#discussion_r543223500



##########
File path: apisix/ssl/router/radixtree_sni.lua
##########
@@ -208,7 +208,9 @@ function _M.match_and_set(api_ctx)
     local sni
     sni, err = ngx_ssl.server_name()
     if type(sni) ~= "string" then
-        return false, "failed to fetch SSL certificate: " .. (err or "not 
found")
+        local advise = "please check if the client requests via IP or uses an 
outdated protocol" ..
+            ". If you need to report an issue, provide a packet capture file 
of the TLS handshake."

Review comment:
       Changed.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to