[keyserver] simplify verifyUserLoggedIn
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F2983816: D13665.id45079.diff
	Wed, Oct 16, 11:25 AM

	F2983704: D13665.diff
	Wed, Oct 16, 10:18 AM

	F2982832: D13665.id45078.diff
	Wed, Oct 16, 6:06 AM

	Unknown Object (File)
	Tue, Oct 15, 11:13 AM

	Unknown Object (File)
	Mon, Oct 14, 7:36 PM

	Unknown Object (File)
	Mon, Oct 14, 2:46 PM

	Unknown Object (File)
	Mon, Oct 14, 1:56 AM

	Unknown Object (File)
	Sat, Oct 12, 11:47 AM

View All 9 Files

Subscribers

tomek

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

No Lint Coverage

Unit

No Test Coverage

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 45020

View Options

keyserver/src/user/login.js

[keyserver] simplify verifyUserLoggedInClosedPublicActions