Page MenuHomePhabricator
Differential D16390

Fix a bad DiffusionCommandEngine parameter from HTTPEngine conversion
ClosedPublic
Actions

Authored by epriestley on Aug 11 2016, 10:18 PM.
Tags
None
Referenced Files
F15466113: D16390.id39420.diff
Thu, Apr 3, 1:43 AM
F15457984: D16390.id39417.diff
Sun, Mar 30, 8:21 PM
F15456160: D16390.diff
Sun, Mar 30, 7:26 AM
F15454868: D16390.diff
Sat, Mar 29, 8:56 PM
F15454363: D16390.diff
Sat, Mar 29, 6:12 PM
F15424056: D16390.id39420.diff
Sat, Mar 22, 8:01 PM
F15418437: D16390.id39417.diff
Thu, Mar 20, 11:20 PM
F15384456: D16390.diff
Fri, Mar 14, 7:57 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Commits
rP39d4e21eec56: Fix a bad DiffusionCommandEngine parameter from HTTPEngine conversion
Summary

I converted this call incorrectly in D16092. We should pass the PhutilURI object, not the string version of it.

Specifically, this resulted in hitting an error like this if a replica needed synchronization:

[2016-08-11 21:22:37] EXCEPTION: (InvalidArgumentException) Argument 1 passed to DiffusionCommandEngine::setURI() must be an instance of PhutilURI, string given, called in...
#0 PhutilErrorHandler::handleError(integer, string, string, integer, array) called at [<phabricator>/src/applications/diffusion/protocol/DiffusionCommandEngine.php:52]
#1 DiffusionCommandEngine::setURI(string) called at [<phabricator>/src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php:601]
...
Test Plan

Clusterized an observed repository, demoted a node, ran bin/repository update Rxxx to update, saw no typehint fatal.

Diff Detail

Repository
rP Phabricator
Branch
uri1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13294
Build 17047: Run Core Tests
Build 17046: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 39417.Aug 11 2016, 10:18 PM
epriestley retitled this revision from to Fix a bad DiffusionCommandEngine parameter from HTTPEngine conversion.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad accepted this revision.Aug 11 2016, 11:38 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 11 2016, 11:38 PM
Closed by commit rP39d4e21eec56: Fix a bad DiffusionCommandEngine parameter from HTTPEngine conversion (authored by epriestley, committed by epriestley). · Explain WhyAug 11 2016, 11:41 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
protocol/
2 lines

Diff 39417

View Options

src/applications/diffusion/protocol/DiffusionRepositoryClusterEngine.php

Loading...
Phabricator · Built by Phacility