Page MenuHomePhabricator
Differential D20026

Add a Duo API future
ClosedPublic
Actions

Authored by epriestley on Jan 23 2019, 9:33 PM.
Tags
None
Referenced Files
F19503837: D20026.diff
Fri, Jan 9, 1:58 AM
F19468792: D20026.diff
Sun, Jan 4, 10:19 AM
F19468790: D20026.diff
Sun, Jan 4, 10:19 AM
F18866832: D20026.id47802.diff
Nov 3 2025, 7:43 PM
F18847351: D20026.diff
Oct 30 2025, 12:10 AM
F18838389: D20026.id.diff
Oct 27 2025, 10:30 AM
F18830435: D20026.diff
Oct 25 2025, 6:45 AM
F18812235: D20026.id47834.diff
Oct 20 2025, 7:52 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13231: Duo MFA Support
Commits
rP069160404fe8: Add a Duo API future
Summary

Depends on D20025. Ref T13231. Although I'm not currently planning to actually upstream a Duo MFA provider, it's probably easiest to put most of the support pieces in the upstream until T5055.

Test Plan

Used a test script to make some (mostly trivial) API calls and got valid results back, so I think the parameter signing is correct.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 23 2019, 9:33 PM
Harbormaster completed remote builds in B21655: Diff 47802.Jan 23 2019, 9:35 PM
epriestley requested review of this revision.Jan 23 2019, 9:35 PM
epriestley added inline comments.Jan 23 2019, 9:36 PM
src/applications/metamta/future/PhabricatorTwilioFuture.php
61

This was a leftover from implementing the thing, setPath() wiped it out. I caught it while COPY/PASTING CODE HA HA HA

amckinley accepted this revision.Jan 24 2019, 7:08 PM
This revision is now accepted and ready to land.Jan 24 2019, 7:08 PM
epriestley added a child revision: D20028: Add CSRF to SMS challenges, and pave the way for more MFA types (including Duo).Jan 24 2019, 10:21 PM
Closed by commit rP069160404fe8: Add a Duo API future (authored by epriestley). · Explain WhyJan 24 2019, 11:10 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
auth/
future/
150 lines
metamta/
future/
2 lines

Diff 47834

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/auth/future/PhabricatorDuoFuture.php

Loading...
View Options

src/applications/metamta/future/PhabricatorTwilioFuture.php

Loading...
Phabricator · Built by Phacility