Page MenuHomePhabricator

Introduce PhutilHTTPEngineExtension, for flexibly compromising SSL
ClosedPublic

Authored by epriestley on Jun 9 2016, 6:45 PM.
Tags
None
Referenced Files
F15458496: D16090.id38715.diff
Mon, Mar 31, 4:04 AM
F15445846: D16090.id38715.diff
Thu, Mar 27, 4:12 PM
F15445756: D16090.diff
Thu, Mar 27, 3:50 PM
F15434000: D16090.diff
Tue, Mar 25, 1:30 AM
F15429866: D16090.diff
Mon, Mar 24, 4:41 AM
F15422828: D16090.id38715.diff
Sat, Mar 22, 10:58 AM
F15417442: D16090.id38715.diff
Thu, Mar 20, 5:01 PM
F15394769: D16090.id38712.diff
Sun, Mar 16, 1:03 AM
Subscribers
None

Details

Summary

Ref T10227. Currently, we have a weird one-off thing in arc for blindly trusting domains, since this was easier than dealing with all the users using self-signed certificates.

Convert this into a modular extension and extend it to support certificates with bad hostnames (maybe plausibly legitimate when connecting to a machine via different interfaces? I guess?) and proxies.

This is the first of three changes which bring support to all of libphutil, arcanist, and Phabricator.

Test Plan
  • Defined a "trust authority" extension, saw it trust/not trust authority.
  • Defined a "proxy" extension, saw requests go through a proxy.
  • Used --trace, saw proxy in trace.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Introduce PhutilHTTPEngineExtension, for flexibly compromising SSL.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 9 2016, 6:57 PM