HomePhabricator

Fix an issue with passing HTTP headers through in proxied cluster requests

Description

Fix an issue with passing HTTP headers through in proxied cluster requests

Summary:
I think this fixes the Mercurial + HTTP cluster issue. PHP adds HTTP_ but we were not stripping it, so we would convert an X-Whatever-Zebra header into an Http-X-Whatever-Zebra header.

I don't think this behavior has changed? So maybe it just never worked? Git is more popular than Mercurial and SSH is easier to configure than HTTP, so it's plausible. I'll keep a careful eye on this when it deploys.

Test Plan:

  • Set up local service-based Mercurial repository.
  • Tried to clone, got similar error to cluster.
  • Applied patch, clean clone.

Reviewers: chad

Reviewed By: chad

Differential Revision: https://secure.phabricator.com/D15660

Details

Provenance
epriestleyAuthored on Apr 8 2016, 4:58 PM
epriestleyPushed on Apr 8 2016, 6:03 PM
Reviewer
chad
Differential Revision
D15660: Fix an issue with passing HTTP headers through in proxied cluster requests
Parents
rP0900ffe9cb3e: Support sorting countdowns by end date
Branches
Unknown
Tags
Unknown
Build Status
Buildable 11603
Build 14505: Run Core Tests