Skip to content
This repository has been archived by the owner on Apr 3, 2019. It is now read-only.

feat(token): implement token and verifyToken endpoints on the auth-server #2931

Closed
wants to merge 1 commit into from
Closed

feat(token): implement token and verifyToken endpoints on the auth-server #2931

wants to merge 1 commit into from

Conversation

vladikoff
Copy link
Contributor

Fixes #2880

@ghost ghost assigned vladikoff Feb 22, 2019
@ghost ghost added the waffle:active label Feb 22, 2019
rfk
rfk reviewed Feb 22, 2019
View reviewed changes
lib/routes/oauth.js

const uid = tokenVerify.user;
const accountRecord = await db.account(uid);

Copy link
Contributor

@rfk rfk Feb 22, 2019 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm aware this is all WIP, just noting that we only want to do the steps below if the token has "oldsync" scope. (Also I think you'll get the notifyDeviceConnected part for free if you actually create a placeholder device record at this point.)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeap, thanks!

@vladikoff vladikoff added the WIP label Feb 22, 2019
This was referenced Feb 27, 2019
Closed
Closed
@vladikoff
Copy link
Contributor Author

Continued in #2985

@vladikoff vladikoff closed this Mar 23, 2019
@ghost ghost removed the waffle:active label Mar 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rfk rfk rfk left review comments

Assignees

@vladikoff vladikoff

Labels
WIP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vladikoff @rfk