Page MenuHomePhabricator

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

Authored by varun on Jun 24 2022, 6:31 AM.
Tags
None
Referenced Files
F3114790: D4350.diff
Thu, Oct 31, 8:44 PM
Unknown Object (File)
Mon, Oct 21, 8:35 PM
Unknown Object (File)
Mon, Oct 21, 6:26 AM
Unknown Object (File)
Fri, Oct 18, 6:09 PM
Unknown Object (File)
Sat, Oct 12, 1:04 PM
Unknown Object (File)
Sat, Oct 12, 12:40 PM
Unknown Object (File)
Thu, Oct 10, 1:44 AM
Unknown Object (File)
Thu, Oct 10, 1:44 AM

Details

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
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun requested review of this revision.Jun 24 2022, 6:35 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

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