[keyserver] simplify verifyUserLoggedIn
ClosedPublic
Actions

Authored by varun on Wed, Oct 9, 12:53 PM.

Details

Reviewers

ashoat
inka

Commits

rCOMMc8a851f8be77: [keyserver] simplify verifyUserLoggedIn

Summary

in the next diff, i split out the auth and save parts of this function into a separate function, so rearranging the code here first

Depends on D13664

Test Plan

read the code carefully

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Wed, Oct 9, 12:53 PM

Herald added a subscriber: tomek. · View Herald TranscriptWed, Oct 9, 12:53 PM

varun added a child revision: D13666: [keyserver] factor out identity auth and saveIdentityInfo from verifyUserLoggedIn.Wed, Oct 9, 12:55 PM

varun edited the summary of this revision. (Show Details)Wed, Oct 9, 1:07 PM

Harbormaster completed remote builds in B32147: Diff 45020.Wed, Oct 9, 1:11 PM

varun requested review of this revision.Wed, Oct 9, 1:11 PM

ashoat accepted this revision.Wed, Oct 9, 5:03 PM

This revision is now accepted and ready to land.Wed, Oct 9, 5:03 PM

Closed by commit rCOMMc8a851f8be77: [keyserver] simplify verifyUserLoggedIn (authored by varun). · Explain WhyThu, Oct 10, 12:07 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMc8a851f8be77: [keyserver] simplify verifyUserLoggedIn.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

user/

5 lines

Diff 45077

View Options

keyserver/src/user/login.js

[keyserver] simplify verifyUserLoggedInClosedPublicActions