Differential D19629

Link to PHP external symbols using HTTPS
AbandonedPublic
Actions

Authored by joshuaspence on Sep 2 2018, 11:06 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Summary

Nowadays, we all use HTTPS.

Test Plan

N/A

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

Tests Skipped

Build Status

Buildable 20725
Build 28172: Run Core Tests
Build 28171: arc lint + arc unit

Event Timeline

joshuaspence created this revision.Sep 2 2018, 11:06 PM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 2 2018, 11:06 PM

Harbormaster completed remote builds in B20725: Diff 46919.Sep 2 2018, 11:08 PM

joshuaspence requested review of this revision.Sep 2 2018, 11:08 PM

Herald added a subscriber: Korvin. · View Herald TranscriptSep 2 2018, 11:08 PM

Almost all of us:

Looks like it depends on which instance you hit: https://www.ssllabs.com/ssltest/analyze.html?d=www.php.net&latest

In D19629#242776, @avivey wrote:

Almost all of us:

Looks like it depends on which instance you hit: https://www.ssllabs.com/ssltest/analyze.html?d=www.php.net&latest

Lol, very well then.

poor php.net

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

symbol/

DiffusionPhpExternalSymbolsSource.php

4 lines

Diff 46919

View Options

src/applications/diffusion/symbol/DiffusionPhpExternalSymbolsSource.php

Link to PHP external symbols using HTTPSAbandonedPublicActions