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
F15503885: D20026.id47802.diff
Mon, Apr 14, 2:08 PM
F15503750: D20026.id.diff
Mon, Apr 14, 12:42 PM
F15502669: D20026.id47834.diff
Mon, Apr 14, 3:03 AM
F15491772: D20026.diff
Sat, Apr 12, 7:22 AM
F15460090: D20026.id47802.diff
Mon, Mar 31, 8:04 PM
F15423460: D20026.id47802.diff
Sat, Mar 22, 3:35 PM
F15422700: D20026.diff
Sat, Mar 22, 10:15 AM
F15389491: D20026.diff
Mar 15 2025, 5:20 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