Paths

Table of Contentst

Differential D4350

[services][identity] helper function to start PAKE login
ClosedPublic
Actions

Authored by varun on Jun 24 2022, 6:31 AM.

Tags

None

Referenced Files

	F3380066: D4350.diff
	Wed, Nov 27, 8:50 PM

	Unknown Object (File)
	Wed, Nov 13, 2:08 AM

	Unknown Object (File)
	Wed, Nov 13, 2:08 AM

	Unknown Object (File)
	Wed, Nov 13, 2:08 AM

	Unknown Object (File)
	Wed, Nov 13, 2:08 AM

	Unknown Object (File)
	Wed, Nov 13, 2:08 AM

	Unknown Object (File)
	Wed, Nov 13, 2:05 AM

	Unknown Object (File)
	Thu, Oct 31, 10:30 PM

Subscribers

Details

Reviewers

tomek
• jim
• karol

Commits

rCOMM0867427da96b: [services][identity] helper function to start PAKE login

Summary

fetch registration data from database, generate challenge for user, return challenge or error

Test Plan

tested with method that calls this helper function

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Jun 24 2022, 6:31 AM

Herald added subscribers: • abosh, atul, • adrian, ashoat. · View Herald TranscriptJun 24 2022, 6:31 AM

varun added a parent revision: D4285: [services][identity] wallet login.Jun 24 2022, 6:32 AM

varun added a child revision: D4351: [services][identity] helper function to finish PAKE login.Jun 24 2022, 6:33 AM

Harbormaster completed remote builds in B9946: Diff 13757.Jun 24 2022, 6:35 AM

varun requested review of this revision.Jun 24 2022, 6:35 AM

varun added 1 blocking reviewer(s): • jim.Jun 24 2022, 8:31 AM

rebase

varun edited parent revisions, added: D4362: [services][identity] update Config struct to hold keypair instead of just private key; removed: D4285: [services][identity] wallet login.Jun 27 2022, 1:01 AM

Harbormaster completed remote builds in B9999: Diff 13816.Jun 27 2022, 1:21 AM

tomek requested changes to this revision.Jun 27 2022, 3:22 AM

tomek added inline comments.

services/identity/src/service.rs
345 ↗	(On Diff #13816)	I'm not sure if it is a good practice in rust, but returning values by parameters is usually really bad for maintainability of the code. We should clearly define a contract where parameters are used to provide the input for a function, and its result is returned. So in this case, `server_login` should become a part of the value that is returned.

This revision now requires changes to proceed.Jun 27 2022, 3:22 AM

address feedback

services/identity/src/service.rs
345 ↗	(On Diff #13816)	makes sense

Harbormaster completed remote builds in B10007: Diff 13842.Jun 27 2022, 8:34 AM

• karol accepted this revision.Jun 28 2022, 2:01 AM

tomek accepted this revision.Jun 28 2022, 4:45 AM

varun removed 1 blocking reviewer(s): • jim.Jun 28 2022, 6:22 AM

This revision is now accepted and ready to land.Jun 28 2022, 6:22 AM

Closed by commit rCOMM0867427da96b: [services][identity] helper function to start PAKE login (authored by varun). · Explain WhyJun 28 2022, 7:12 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMM0867427da96b: [services][identity] helper function to start PAKE login.

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

87 lines

Diff 13895

services/identity/src/service.rs

Loading...