Skip to content

Add "human detection" to voice OTP phone call#768

Closed
zachmargolis wants to merge 1 commit intomasterfrom
margolis-twilio-human-detection
Closed

Add "human detection" to voice OTP phone call#768
zachmargolis wants to merge 1 commit intomasterfrom
margolis-twilio-human-detection

Conversation

@zachmargolis
Copy link
Contributor

Why:
So we don't read our OTPs into people's voicemails

How:
Replace intial message that reads the code with one that
prompts users to press 1 to hear the code


Expands on #767 to add human detection

@zachmargolis zachmargolis force-pushed the margolis-twilio-human-detection branch from fa26ea0 to b554f76 Compare November 25, 2016 15:42
**Why**:
So we don't read our OTPs into people's voicemails

**How**:
Replace intial message that reads the code with one that
prompts users to press 1 to hear the code
@monfresh
Copy link
Contributor

I'm testing this right now, and the voice message keeps playing in a loop:

Your login.gov one time passcode is [code], again, your passcode is [code]. 
Press 1 to repeat your passcode. 
[silence for a few seconds].

This repeats seemingly ad infinitum.

Is this expected?

@zachmargolis
Copy link
Contributor Author

@monfresh sort of expected? Unfortunately the Twimlet code for menus includes a <Redirect> tag after the menu which will cause the whole script to loop after a timeout (see example menu xml)

We'll have to sort out the infinite looping with #761 when we can self-host the TwiML and have more control over menu behavior.

Based on some discussion with @usdsclarax in https://github.com/18F/identity-private/issues/604, I'm going to close this issue since our current thinking is we don't need/want human detection.

@zachmargolis zachmargolis deleted the margolis-twilio-human-detection branch April 6, 2017 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants