Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Comparing changes

Choose two branches to see what's changed or to start a new pull request. If you need to, you can also compare across forks.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also compare across forks.
...
Checking mergeability… Don't worry, you can still create the pull request.
  • 2 commits
  • 1 file changed
  • 0 commit comments
  • 2 contributors
Commits on Aug 10, 2012
@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.
4b3b136
Commits on Aug 13, 2012
@konstantin-dzreev konstantin-dzreev Merge pull request #13 from litvinovsky/purple_al_12_9__13307_remove_…
…ssl_logging_stuff_release4.3_gw_hotfix

refs #13307 - remove ssl logging stuff due it's useless
c03daef
Showing with 0 additions and 6 deletions.
  1. +0 −6 lib/right_http_connection.rb
View
6 lib/right_http_connection.rb
@@ -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

No commit comments for this range

Something went wrong with that request. Please try again.