Partially addresses ENG-6545.
Added a function to clear user ID and access token from metadata table. It's going to be called when service authentication fails.
Details
Details
Called this function manually and verified in TablePlus that these rows were removed. Calling verifyUserLoggedIn() creates them again.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| keyserver/src/user/identity.js | ||
|---|---|---|
| 131 ↗ | (On Diff #45864) | Nit: this query will crash if passed an empty array. We don't do that currently, but I've seen this accidentally happen enough times that I usually try to cover it |
| 136 ↗ | (On Diff #45864) | I think it would be better to return Promise<void>. The QueryResult type has some anys and you never seem to use it from any of the callsites of clearMetadata currently in this stack |