Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

refs #13307 - remove ssl logging stuff due it's useless

Initially we decide to remove ssl logging because there are too
many logs connected with that. We don't need to log this stuff,
because if there is problem with authorization then the server
will return appropriate status code in the response.
  • Loading branch information...
commit 82e29a5b6a56f81c6cfc063d06c0325b36fe63b4 1 parent 72c4b2d
Alexander Litvinovsky litvinovsky authored

Showing 1 changed file with 0 additions and 6 deletions. Show diff stats Hide diff stats

  1. +0 6 lib/right_http_connection.rb
6 lib/right_http_connection.rb
@@ -310,12 +310,6 @@ def start(request_params)
310 310 # List of error codes: http://www.openssl.org/docs/apps/verify.html
311 311 code = x509_store_ctx.error
312 312 msg = x509_store_ctx.error_string
313   - #debugger
314   - if code == 0
315   - @logger.info "##### Certificate verification passed. #####"
316   - else
317   - @logger.warn("##### #{@server} certificate verify failed: #{msg}")
318   - end
319 313 if request_params[:fail_if_ca_mismatch] && code != 0
320 314 false
321 315 else

0 comments on commit 82e29a5

Please sign in to comment.
Something went wrong with that request. Please try again.