Make WordPress Core

Opened 12 years ago

Closed 11 years ago

#9078 closed enhancement (fixed)

Inclusion of SSL Blacklisting in HTTP Transport Tests

Reported by: sivel Owned by:
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.8
Component: HTTP API Keywords: has-patch tested commit
Focuses: Cc:


We are currently doing ssl blacklisting for certain transports by tacking on some logic to the if statements in the transport selection.

I'd like to add the ssl blacklisting to the test functions an update it a bit.

Since we are blacklisting transports that cannot do ssl verification, we should allow these transports to accept ssl if https_ssl_verify or https_local_ssl_verify are set to false by filters or if the call is made using 'sslverify' => 'false'

fsockopen can still not handle invalid ssl certs so it is ssl blacklisted no matter what.

Attachments (5)

9078.diff (7.2 KB) - added by sivel 12 years ago.
9078.2.diff (7.2 KB) - added by sivel 12 years ago.
9078.3.diff (6.1 KB) - added by sivel 12 years ago.
9078.4.diff (5.0 KB) - added by sivel 11 years ago.
list_transports.php (1.8 KB) - added by sivel 11 years ago.
Updated List Transports plugin

Download all attachments as: .zip

Change History (11)

12 years ago

#1 @sivel
12 years ago

I'm also going to roll the use_*_transport filters into the tests as well.

One benefit of moving everything into the tests is that we will have to pass the $args array to the test. This will enable us in the long run to build out more extensive tests.

12 years ago

#2 @sivel
12 years ago

If anyone would like to help in testing these changes it would be greatly appreciated. Report back here any issues you may run into.

12 years ago

11 years ago

#3 @sivel
11 years ago

  • Cc matt@… added
  • Keywords tested commit added; needs-testing removed

Updates against recent updates on http.php

11 years ago

Updated List Transports plugin

#4 @sivel
11 years ago

I updated the list transports plugin written by Ryan to show both POST and GET transports for both HTTP and HTTPS requests.

#5 @sivel
11 years ago

  • Component changed from General to HTTP
  • Owner anonymous deleted

#6 @westi
11 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [10642]) Refactor the ssl blacklisting support info the individual test functions. Fixes #9078 props sivel.

Note: See TracTickets for help on using tickets.