Page MenuHomePhabricator
Differential D5739

[keyserver] APIs for finishing client side of PAKE login
ClosedPublic
Actions

Authored by varun on Nov 28 2022, 7:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 22, 8:01 AM
Unknown Object (File)
Fri, Nov 22, 8:01 AM
Unknown Object (File)
Fri, Nov 22, 8:01 AM
Unknown Object (File)
Fri, Nov 22, 8:01 AM
Unknown Object (File)
Fri, Nov 22, 7:59 AM
Unknown Object (File)
Fri, Nov 22, 7:45 AM
Unknown Object (File)
Tue, Nov 12, 9:38 AM
Unknown Object (File)
Tue, Nov 12, 1:21 AM
View All Files
Subscribers
ashoat
bartek

Details

Reviewers
tomek
• jon
bartek
Commits
rCOMMcee46d8c4a98: [keyserver] APIs for finishing client side of PAKE login
Summary

using Neon to surface the Rust opaque-ke "client login finish" API and getters to Node.js

Test Plan

called the new APIs from a Node module and confirmed the results

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun created this revision.Nov 28 2022, 7:51 AM
Herald added a subscriber: ashoat. · View Herald TranscriptNov 28 2022, 7:51 AM
varun added a parent revision: D5738: [keyserver] APIs for starting client side of PAKE login.Nov 28 2022, 7:57 AM
Harbormaster completed remote builds in B13791: Diff 18882.Nov 28 2022, 8:05 AM
varun requested review of this revision.Nov 28 2022, 8:05 AM
atul added a reviewer: bartek.Nov 28 2022, 2:53 PM
atul added a subscriber: bartek.

(Think it probably makes sense to add @bartek to your Rust diffs even if he doesn't have full context on identity stuff? CC @ashoat @tomek who probably have a better idea)

atul resigned from this revision.Nov 28 2022, 3:52 PM

Don't have context + enough rust knowledge here

tomek added inline comments.Nov 29 2022, 7:28 AM
keyserver/addons/opaque-ke-node/src/lib.rs
212–216 ↗(On Diff #18882)

Please avoid using shadowing

varun updated this revision to Diff 18995.Nov 29 2022, 11:33 PM

change variable names to avoid shadowing, change js functions to camelCase

varun edited reviewers, added: Restricted Owners Package; removed: tomek, atul, bartek.Nov 29 2022, 11:34 PM
Harbormaster failed remote builds in B13871: Diff 18995!Nov 29 2022, 11:34 PM
varun edited reviewers, added: tomek, • jon, bartek; removed: Restricted Owners Package.Nov 29 2022, 11:34 PM
varun updated this revision to Diff 19000.Nov 29 2022, 11:49 PM

rebase

Harbormaster completed remote builds in B13876: Diff 19000.Nov 30 2022, 12:01 AM
tomek accepted this revision.Nov 30 2022, 5:33 AM
bartek accepted this revision.Nov 30 2022, 6:04 AM
• jon accepted this revision.Nov 30 2022, 11:44 AM
This revision is now accepted and ready to land.Nov 30 2022, 11:44 AM
Closed by commit rCOMMcee46d8c4a98: [keyserver] APIs for finishing client side of PAKE login (authored by varun). · Explain WhyNov 30 2022, 1:04 PM
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMMcee46d8c4a98: [keyserver] APIs for finishing client side of PAKE login.

Revision Contents
Changeset List

PathSize
keyserver/
addons/
opaque-ke-node/
src/
64 lines

Diff 18995

View Options

keyserver/addons/opaque-ke-node/src/lib.rs

Loading...