Skip to content
This repository has been archived by the owner on Feb 21, 2020. It is now read-only.

Add nonce #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add nonce #11

wants to merge 1 commit into from

Conversation

fabien0102
Copy link

So, as I said into #9, we need to add nonce to have this token_id working without downgrade security into auth0 panel 😉

Note: I have tested on my own playground

@fabien0102 fabien0102 mentioned this pull request Oct 5, 2017
Closed
@ice-chillios
Copy link

@charlesvinette Could we merge this? Seems to be requirement for auth0 callbacks :)

@charlesvinette
Copy link
Contributor

Hey guys! Thanks again for taking the time to help us out with the example. I will have a look later today and get back to you. Have a good day!

@fabien0102
Copy link
Author

Conflicts… 😢 I personally don't use expo with auth-0 for instant. BTW I can, of course, rebase this branch if you want, but only if this one have the intention to merge this one ^^ Otherwise, I will just close this old PR. Your choice 😉

cc @dsznajder @charlesvinette

@charlesvinette
Copy link
Contributor

@fabien0102 Of course, I would appreciate it! Feel free to do the change, we will review and merge. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ice-chillios ice-chillios ice-chillios approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fabien0102 @ice-chillios @charlesvinette