Page MenuHomePhabricator

[keyserver] Add a function that synchronizes keyserver's invite links with blobs
ClosedPublic

Authored by tomek on Dec 6 2023, 6:13 AM.
Tags
None
Referenced Files
F1799188: D10205.diff
Mon, May 20, 3:28 AM
Unknown Object (File)
Sun, May 12, 5:19 PM
Unknown Object (File)
Sun, May 12, 2:33 AM
Unknown Object (File)
Mar 6 2024, 9:39 AM
Unknown Object (File)
Mar 6 2024, 9:39 AM
Unknown Object (File)
Mar 6 2024, 9:39 AM
Unknown Object (File)
Mar 6 2024, 9:39 AM
Unknown Object (File)
Mar 6 2024, 9:39 AM
Subscribers

Details

Summary

In this script we try to create blobs for links where a holder is missing or a holder doesn't point to an existing blob. This function will be used in a migration and in a script that periodically synchronizes data.

Depends on D10150

Test Plan

Run the function as a script and check if it creates blobs for all the links that don't have them.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek requested review of this revision.Dec 6 2023, 6:31 AM
keyserver/src/creators/invite-link-creator.js
202 ↗(On Diff #34336)

This should not be an async function, since it does not use the await keyword. This causes an unnecessary step where we wrap a Promise in another Promise

rohan requested changes to this revision.Dec 12 2023, 6:32 AM

Just requesting changes for the inline comment so it goes back in your queue

This revision now requires changes to proceed.Dec 12 2023, 6:32 AM
tomek marked an inline comment as done.

Rebase

This revision is now accepted and ready to land.Feb 6 2024, 2:42 AM