Page MenuHomeDevCentral
Differential D484

RFC 7239 compliant preamble script for Phabricator
ClosedPublic
Actions

Authored by dereckson on Jul 10 2016, 23:44.
Tags
Referenced Files
F3765894: D484.id1178.diff
Fri, Nov 22, 15:03
F3765686: D484.id.diff
Fri, Nov 22, 13:45
F3764879: D484.diff
Fri, Nov 22, 08:58
Unknown Object (File)
Mon, Nov 18, 08:12
Unknown Object (File)
Sat, Nov 16, 01:08
Unknown Object (File)
Wed, Nov 13, 00:54
Unknown Object (File)
Tue, Nov 12, 14:15
Unknown Object (File)
Mon, Nov 11, 14:49
View All Files
Subscribers
Nasqueron Docker deployment squad

Details

Reviewers
Sandlayth
Maniphest Tasks
T915: Preamble script for Phabricator
Commits
rDPHAB892cd0a35934: RFC 7239 compliant preamble script for Phabricator
Summary

Fix headers to allow two things:

  • correct detection of http/https
  • remote address is use instead of Docker engine address

Reference:

Fixes T915.

Test Plan

Tested on https://devcentral.nasqueron.org/

Diff Detail

Repository
rDPHAB Docker image for Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 1178.Jul 10 2016, 23:44
dereckson retitled this revision from to Preamble script for Phabricator.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: Sandlayth.
dereckson added a task: T915: Preamble script for Phabricator.
Herald added a project: Docker images. · View Herald TranscriptJul 10 2016, 23:44
Herald added a subscriber: Nasqueron Docker deployment squad. · View Herald Transcript
Sandlayth accepted this revision.Jul 11 2016, 06:59
Sandlayth edited edge metadata.
This revision is now accepted and ready to land.Jul 11 2016, 06:59
dereckson retitled this revision from Preamble script for Phabricator to RFC 7239 compliant preamble script for Phabricator .Jul 11 2016, 11:03
dereckson updated this object.
dereckson edited edge metadata.
Closed by commit rDPHAB892cd0a35934: RFC 7239 compliant preamble script for Phabricator (authored by dereckson, committed by dereckson). · Explain WhyJul 11 2016, 11:07
This revision was automatically updated to reflect the committed changes.
dereckson mentioned this in T275: Nginx proxies request with 127.0.0.1 as IP address.Feb 14 2020, 17:42

Revision Contents
Changeset List

PathSize
files/
opt/
phabricator/
support/
15 lines

Diff 1197

View Options

files/opt/phabricator/support/preamble.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator