-
-
Notifications
You must be signed in to change notification settings - Fork 98
[WIP] feat: Migration for PGP-contacts #6818
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
c5b26a0
to
6a95b3e
Compare
3c4255a
to
d912d59
Compare
781a76b
to
f3fb5a2
Compare
f5a2a18
to
3cdd704
Compare
c1456d6
to
c47c105
Compare
c47c105
to
cb02753
Compare
5bc12ea
to
9b16d25
Compare
During migration DC community chat shrinked from 114 members (not past members) to 110 members. The log has these lines for the community chat:
This deleted some real members, likely because there are past members without the key or with the same key as they did AEAP in the past. Contact 35 is migrated into 928. Then according to the log we try to migrate 298 (this is the current email address), fail, remove 928 and try again. Probably at this point we have the correct contact with correct address in the chat. Then we try to migrate 298, which is a past member. We fail, so we remove correct 928 and migrate 298 into 928. In the end 928 has the wrong email address. We should have migrated 218, but somehow got contact 905 in the chat. |
54bab1e
to
f70f358
Compare
074f5f2
to
21ea8af
Compare
c156eca
to
ec7c421
Compare
…ent addr If a contact did AEAP in the past, then it can happen that there are multiple contacts with the same email address but different keys. In this case, we should make sure that we don't send messages to a different email address all of a sudden, and that there aren't two 1:1 chats with the same contact. Therefore, if the PGP contact has a different email address than the email contact, we keep the email contact in 1:1 chats.
21ea8af
to
63fa13a
Compare
Merged migration into PGP-contacts PR |
Based on #6796
TODO:
To decide:
verifier
column, and define this to mean "unknown introducer".