[Identity] Remove static lifetime constraint on DBItemError
ClosedPublic
Actions

Authored by • jon on Jul 25 2023, 10:44 AM.

Details

Reviewers

bartek
varun

Commits

rCOMM83836ecf16b7: [Identity] Remove static lifetime constraint on DBItemError

Summary

In order to support device maps, we can't assume
that all attribute keys will be known staticly.

In support of https://linear.app/comm/issue/ENG-4445

Depends on D8607

Test Plan

cd identity && cargo build

or run any integration tests

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• jon created this revision.Jul 25 2023, 10:44 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 25 2023, 10:44 AM

Harbormaster completed remote builds in B21194: Diff 29030.Jul 25 2023, 11:11 AM

• jon requested review of this revision.Jul 25 2023, 11:11 AM

varun accepted this revision.Jul 25 2023, 12:41 PM

This revision is now accepted and ready to land.Jul 25 2023, 12:41 PM

• jon added a child revision: D8642: Allow for devices to register as a keyserver.Jul 26 2023, 5:37 PM

Closed by commit rCOMM83836ecf16b7: [Identity] Remove static lifetime constraint on DBItemError (authored by • jon). · Explain WhyAug 8 2023, 9:01 AM

This revision was automatically updated to reflect the committed changes.

• jon added a commit: rCOMM83836ecf16b7: [Identity] Remove static lifetime constraint on DBItemError.

• jon mentioned this in D8761: [identity] db method to get inbound keys for user.Aug 8 2023, 9:02 AM

bartek mentioned this in D8787: [services-lib] Make DB attribute names non-static.Aug 10 2023, 3:33 PM

bartek mentioned this in rCOMMff30037d57c5: [services-lib] Make DB attribute names non-static.Aug 16 2023, 7:25 AM

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

database.rs

32 lines

error.rs

2 lines

Diff 29030

View Options

services/identity/src/database.rs