Page MenuHomePhabricator
Differential D11761

[lib] Extract useRawKeyserverAuth
ClosedPublic
Actions

Authored by ashoat on Wed, Apr 24, 12:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 30, 6:45 PM
Unknown Object (File)
Mon, Apr 29, 11:59 AM
Unknown Object (File)
Sun, Apr 28, 11:15 AM
Unknown Object (File)
Wed, Apr 24, 2:39 PM
Unknown Object (File)
Wed, Apr 24, 2:38 PM
Unknown Object (File)
Wed, Apr 24, 2:35 PM
Subscribers
None

Details

Reviewers
tomek
inka
Commits
rCOMMd8130c105d12: [lib] Extract useRawKeyserverAuth
Summary

This addresses ENG-7668, which is explained more in its parent ENG-7667.

Test Plan

This whole stack was tested with the following steps:

  1. On native, attempting to log in with an ETH account that hasn’t been registered yet
  2. On native, attemping to register with an ETH account that has already been registered
  3. On native, log in with an ETH account that has already been registered
  4. On native, register with an ETH account that hasn’t already been registered
  5. On native, register with a password account
  6. On native, log in with a password account
  7. On web, attempting to log in with an ETH account that hasn’t been registered yet
  8. On web, log in with an ETH account that has already been registered
  9. On web, log in with a password account

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat created this revision.Wed, Apr 24, 12:39 PM
ashoat added a child revision: D11762: [lib] Don't try authing to authoritative keyserver from KeyserverConnectionHandler.
Harbormaster completed remote builds in B28437: Diff 39447.Wed, Apr 24, 12:56 PM
ashoat requested review of this revision.Wed, Apr 24, 12:56 PM
inka accepted this revision.Thu, Apr 25, 2:22 AM
inka added inline comments.
lib/keyserver-conn/keyserver-auth.js
28 ↗(On Diff #39447)

Once D11758 is landed, we should probably use permissionsAndAuthRelatedRequestTimeout

This revision is now accepted and ready to land.Thu, Apr 25, 2:22 AM
ashoat added inline comments.Thu, Apr 25, 5:45 AM
lib/keyserver-conn/keyserver-auth.js
28 ↗(On Diff #39447)

I don't think so. This isn't a timeout for the request; rather, it's a delay between retries

Closed by commit rCOMMd8130c105d12: [lib] Extract useRawKeyserverAuth (authored by ashoat). · Explain WhyFri, Apr 26, 8:18 AM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMMd8130c105d12: [lib] Extract useRawKeyserverAuth.

Revision Contents
Changeset List

PathSize
lib/
keyserver-conn/
174 lines
176 lines

Diff 39552

View Options

lib/keyserver-conn/keyserver-auth.js

Loading...
View Options

lib/keyserver-conn/keyserver-connection-handler.js

Loading...