Page MenuHomePhabricator
Differential D20529

Support timeouts in OAuth1Future
ClosedPublic
Actions

Authored by epriestley on May 20 2019, 6:39 PM.
Tags
None
Referenced Files
F15473263: D20529.id48958.diff
Sat, Apr 5, 8:47 PM
F15471160: D20529.id.diff
Sat, Apr 5, 3:41 AM
F15469546: D20529.diff
Fri, Apr 4, 2:27 PM
F15466901: D20529.id48973.diff
Thu, Apr 3, 10:38 AM
F15425466: D20529.id48973.diff
Sun, Mar 23, 4:46 AM
F15425221: D20529.id.diff
Sun, Mar 23, 3:15 AM
F15403109: D20529.diff
Tue, Mar 18, 1:56 AM
F15403083: D20529.id48973.diff
Tue, Mar 18, 1:51 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13285: Service failures in JIRA can cascade into service failures in Phabricator
T13291: Implement a "{src ...}" rule in Remarkup for referencing source files in Diffusion
Commits
rPHU07e92da74e9a: Support timeouts in OAuth1Future
Summary

Ref T13291. Ref T13285. Add "setTimeout()" to OAuth1Future to support adding timeouts to Doorkeeper OAuth requests, so we're less likely to cascade into explosive failure if a linked JIRA instance dies.

Test Plan

See next change.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 20 2019, 6:39 PM
Harbormaster completed remote builds in B22832: Diff 48956.May 20 2019, 6:39 PM
epriestley requested review of this revision.May 20 2019, 6:39 PM
epriestley updated this revision to Diff 48958.May 20 2019, 6:45 PM
  • Also add support to the "AsanaFuture", which is a separate future.
Harbormaster completed remote builds in B22834: Diff 48958.May 20 2019, 6:46 PM
amckinley accepted this revision.May 21 2019, 5:44 PM
This revision is now accepted and ready to land.May 21 2019, 5:44 PM
Closed by commit rPHU07e92da74e9a: Support timeouts in OAuth1Future (authored by epriestley). · Explain WhyMay 21 2019, 7:13 PM
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPHU07e92da74e9a: Support timeouts in OAuth1Future.

Revision Contents
Changeset List

PathSize
src/
future/
asana/
15 lines
oauth/
15 lines

Diff 48973

View Options

src/future/asana/PhutilAsanaFuture.php

Loading...
View Options

src/future/oauth/PhutilOAuth1Future.php

Loading...
Phabricator · Built by Phacility