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
F12806252: D14200.diff
Wed, Mar 27, 6:04 PM
Unknown Object (File)
Sun, Mar 24, 11:59 PM
Unknown Object (File)
Sun, Mar 24, 10:10 PM
Unknown Object (File)
Sun, Mar 24, 9:45 PM
Unknown Object (File)
Sun, Mar 24, 8:02 PM
Unknown Object (File)
Thu, Mar 21, 10:03 PM
Unknown Object (File)
Sat, Mar 16, 3:12 AM
Unknown Object (File)
Mon, Mar 4, 4:36 PM
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