Page MenuHomePhabricator
Differential D16241

Redirect https blogs
ClosedPublic
Actions

Authored by chad on Jul 6 2016, 8:33 PM.
Tags
None
Referenced Files
F18766837: D16241.id39070.diff
Tue, Oct 7, 6:50 PM
F18726999: D16241.diff
Tue, Sep 30, 7:27 AM
F18680901: D16241.diff
Fri, Sep 26, 3:24 AM
F18654388: D16241.diff
Mon, Sep 22, 7:54 AM
F18652783: D16241.id.diff
Sun, Sep 21, 9:31 AM
F18613108: D16241.diff
Sun, Sep 14, 10:49 AM
F18550192: D16241.diff
Sep 8 2025, 10:07 AM
F18550014: D16241.diff
Sep 8 2025, 9:57 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9360: Unbeta Phame
Commits
rPee460b4f1b4c: Redirect https blogs
Summary

Ref T9360, forces https if we say the blog is https.

Test Plan

Fake an https, get redirected.

Diff Detail

Repository
rP Phabricator
Branch
phame-https (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12951
Build 16532: Run Core Tests
Build 16531: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 39070.Jul 6 2016, 8:33 PM
chad retitled this revision from to Redirect https blogs.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T9360: Unbeta Phame.
Herald added a subscriber: Korvin. · View Herald TranscriptJul 6 2016, 8:33 PM
epriestley requested changes to this revision.Jul 6 2016, 11:25 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/phame/site/PhameBlogSite.php
23

I'd expect this to need a $full_uri->getProtocol() here somewhere, the bare $full_uri should never be "https".

This revision now requires changes to proceed.Jul 6 2016, 11:25 PM
chad added a comment.Jul 6 2016, 11:52 PM

I knew I was going to be busted on that test plan. Haha. FWIW I got stumped by the parent Domain bug and never went back to this after fixing.

epriestley added a comment.Jul 6 2016, 11:56 PM

I can kick the tires locally before I accept, I have it sort-of-configured locally.

chad updated this revision to Diff 39075.Jul 7 2016, 12:19 AM
chad edited edge metadata.
  • check protocol, rerun test, see redirect
epriestley accepted this revision.Jul 7 2016, 12:20 AM
epriestley edited edge metadata.

Cool, this works properly in my only-sort-of-fake local environment:

Screen Shot 2016-07-06 at 5.20.08 PM.png (1×1 px, 344 KB)

This revision is now accepted and ready to land.Jul 7 2016, 12:20 AM
Closed by commit rPee460b4f1b4c: Redirect https blogs (authored by chad, committed by chad). · Explain WhyJul 7 2016, 12:22 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
phame/
site/
7 lines

Diff 39075

View Options

src/applications/phame/site/PhameBlogSite.php

Loading...
Phabricator · Built by Phacility