Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

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

Merged
merged 1 commit into from

2 participants

Alexander Litvinovsky Konstantin Dzreev
Alexander Litvinovsky

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.

Alexander Litvinovsky litvinovsky 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.
82e29a5
Konstantin Dzreev konstantin-dzreev merged commit 779706e into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 13, 2012
  1. Alexander Litvinovsky

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

    litvinovsky authored
    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.
This page is out of date. Refresh to see the latest.
Showing with 0 additions and 6 deletions.
  1. +0 −6 lib/right_http_connection.rb
6 lib/right_http_connection.rb
View
@@ -310,12 +310,6 @@ def start(request_params)
# List of error codes: http://www.openssl.org/docs/apps/verify.html
code = x509_store_ctx.error
msg = x509_store_ctx.error_string
- #debugger
- if code == 0
- @logger.info "##### Certificate verification passed. #####"
- else
- @logger.warn("##### #{@server} certificate verify failed: #{msg}")
- end
if request_params[:fail_if_ca_mismatch] && code != 0
false
else
Something went wrong with that request. Please try again.