Page MenuHomePhabricator

Allow 201 status code as a response for harbormaster build plans.
Needs RevisionPublic

Authored by ybv on Sep 30 2015, 7:38 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 23, 5:26 AM
Unknown Object (File)
Fri, Jun 17, 4:00 PM
Unknown Object (File)
Sun, Jun 12, 3:50 AM
Unknown Object (File)
Sat, Jun 11, 4:40 PM
Unknown Object (File)
Sat, Jun 11, 8:59 AM
Unknown Object (File)
Sat, Jun 11, 6:46 AM
Unknown Object (File)
Sat, Jun 11, 6:46 AM
Unknown Object (File)
Mon, Jun 6, 8:05 AM
Subscribers

Details

Summary

Allow 201 status code as a response for harbormaster build plans.

Test Plan

None.

Diff Detail

Repository
rP Phabricator
Branch
allow-201-hm
Lint
Lint Passed
Unit
Tests Skipped
Build Status
Buildable 8106
Build 9238: [Placeholder Plan] Wait for 30 Seconds
Build 9237: arc lint + arc unit

Event Timeline

ybv retitled this revision from to Allow 201 status code as a response for harbormaster build plans..
ybv updated this object.
ybv edited the test plan for this revision. (Show Details)
eadler added inline comments.
src/applications/harbormaster/step/HarbormasterHTTPRequestBuildStepImplementation.php
92

why not allow 2xx in general?

epriestley added a reviewer: epriestley.

Obsolete in HEAD.

This revision now requires changes to proceed.Nov 2 2015, 4:44 PM