Page MenuHomePhabricator

let arc patch authenticate in more cases
Needs ReviewPublic

Authored by eadler on Aug 5 2016, 9:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jun 25, 12:45 PM
Unknown Object (File)
Tue, Jun 7, 11:32 PM
Unknown Object (File)
May 30 2022, 2:00 PM
Unknown Object (File)
May 29 2022, 9:08 PM
Unknown Object (File)
Mar 7 2017, 8:13 AM
Unknown Object (File)
Feb 25 2017, 7:52 PM
Unknown Object (File)
Feb 25 2017, 7:52 PM
Unknown Object (File)
Feb 8 2017, 12:50 AM

Details

Reviewers
magnetik
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T11434: Arc patch fails: EXCEPTION: (ConduitClientException) ERR-INVALID-AUTH: Unknown Error.
Commits
Restricted Diffusion Commit
Summary

fixes T11434

Test Plan

ran ./bin/arc patch against secure. with an empty arcrc
also tried running locally (though its harder since I need to fake a
patch)

Diff Detail

Repository
rARC Arcanist
Branch
eax/T11434 (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13251
Build 16985: Run Core Tests
Build 16984: arc lint + arc unit

Event Timeline

eadler retitled this revision from to fix arc patch.
eadler updated this object.
eadler edited the test plan for this revision. (Show Details)
eadler added a reviewer: magnetik.
magnetik edited edge metadata.

Good for me

avivey retitled this revision from fix arc patch to let arc patch authenticate in more cases.Aug 5 2016, 6:51 PM
avivey edited edge metadata.

LGTM... We don't actually list all the possible errors anywhere, do we?

Is there anything I can do to make this move forward?

Is there anything I can do to make this move forward?

No.

Rtls added a commit: Restricted Diffusion Commit.Nov 24 2016, 6:18 AM