Have identity service issue a request to users when refresh keys
when the key count goes below the threshold.
https://linear.app/comm/issue/ENG-4446
Depends on D8723
Details
Details
Integration tests are in next diff
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| services/identity/src/database.rs | ||
|---|---|---|
| 388–389 ↗ | (On Diff #29584) | Do we have to wait for these? Can it be done concurrently, without blocking the single onetime key acquisition? |
| services/identity/src/database.rs | ||
|---|---|---|
| 388–389 ↗ | (On Diff #29584) | It needs to be awaited (futures that aren't awaited never get executed). But could probably do something like tokio::spawn to move it to a different thread. It my plan originally, but I just wanted something easier to reason about while implementing this. The other item I thought of, and is probably a better solution, would be to introduce a mpsc channel to a dedicated tunnelbroker channel. It would also avoid the overhead of a connection for every request. |
| 390 ↗ | (On Diff #29584) | This is just to avoid the "Result not inspected" warning. It will emit the error in logs. |
| services/identity/src/database.rs | ||
|---|---|---|
| 387–390 ↗ | (On Diff #29584) | Yep, let's try tokio::spawn and see if it works
Good idea. This can be done as a follow-up I think. |
Comment Actions
Apply suggestions
| services/identity/src/constants.rs | ||
|---|---|---|
| 131–133 ↗ | (On Diff #29584) |
It was pretty informal, I think a one-on-one or some crypto sync. Don't think it was ever documented. I just need something though to start with. We can discuss this as part of a crypto sync if you do have concerns. I don't think I'm informed enough to know what the correct numbers would be.
This is address treewide in D8817 |
| services/identity/src/database.rs | ||
| 379–385 ↗ | (On Diff #29584) | After realizing that my let-else issues were related to not running rustup update since February, I decided to use @bartek suggestion. |