Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(14)

Issue 225330043: Suppress all SSL errors. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 6 months ago by erikchen1
Modified:
11 years, 6 months ago
Reviewers:
nednguyen
Base URL:
https://github.com/chromium/web-page-replay@master
Visibility:
Public.

Description

Suppress all SSL errors. By default, each SSL certificate error generates a full traceback. This is pretty pointless, since we know that all SSL requests are going to have certificate errors. BUG= R=nednguyen@google.com Committed: 2fdf47ef7df4fc3dc2d88f5bdc11c7749e37dbdf

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -0 lines) Patch
M httpproxy.py View 5 chunks +22 lines, -0 lines 0 comments Download

Messages

Total messages: 3
erikchen1
nednguyen: Please review.
11 years, 6 months ago (2015-04-10 02:19:29 UTC) #1
nednguyen
lgtm
11 years, 6 months ago (2015-04-10 19:10:11 UTC) #2
erikchen1
11 years, 6 months ago (2015-04-10 22:10:50 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
2fdf47ef7df4fc3dc2d88f5bdc11c7749e37dbdf (presubmit successful).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b