Page MenuHomePhabricator
Differential D11698

[lib] Move uploadBlob to lib/keyserver-conn
ClosedPublic
Actions

Authored by ashoat on Apr 19 2024, 8:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 3, 9:11 AM
Unknown Object (File)
Sun, Nov 3, 9:11 AM
Unknown Object (File)
Sun, Nov 3, 9:11 AM
Unknown Object (File)
Sun, Nov 3, 9:11 AM
Unknown Object (File)
Oct 15 2024, 11:36 AM
Unknown Object (File)
Oct 12 2024, 10:38 PM
Unknown Object (File)
Oct 2 2024, 11:55 AM
Unknown Object (File)
Sep 14 2024, 11:47 PM
View All 40 Files
Subscribers
None

Details

Reviewers
tomek
inka
Commits
rCOMM34e9d5b1913b: [lib] Move uploadBlob to lib/keyserver-conn
Summary

Now that callSingleKeyserverEndpoint has been moved, it makes sense to move uploadBlob too, since it's primarily used from callSingleKeyserverEndpoint.

Depends on D11697

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat created this revision.Apr 19 2024, 8:33 AM
ashoat added a child revision: D11699: [lib][web] Move useKeyserverCall to lib/keyserver-conn.
Harbormaster completed remote builds in B28327: Diff 39282.Apr 19 2024, 8:56 AM
ashoat requested review of this revision.Apr 19 2024, 8:56 AM
tomek accepted this revision.Apr 19 2024, 10:11 AM
tomek added inline comments.
lib/keyserver-conn/upload-blob.js
12 ↗(On Diff #39282)

We can consider renaming it to avoid confusion with blobs from blob service.

This revision is now accepted and ready to land.Apr 19 2024, 10:11 AM
ashoat added inline comments.Apr 19 2024, 11:08 AM
lib/keyserver-conn/upload-blob.js
12 ↗(On Diff #39282)

Okay, but I'll do this in a separate diff as it's somewhat involved, and I think it's best to leave this diff as a "pure move"

Closed by commit rCOMM34e9d5b1913b: [lib] Move uploadBlob to lib/keyserver-conn (authored by ashoat). · Explain WhyApr 19 2024, 11:12 AM
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMM34e9d5b1913b: [lib] Move uploadBlob to lib/keyserver-conn.
ashoat added inline comments.Apr 19 2024, 11:49 AM
lib/keyserver-conn/upload-blob.js
12 ↗(On Diff #39282)

D11703

Revision Contents
Changeset List

PathSize
lib/
actions/
2 lines
keyserver-conn/
2 lines
keyserver-conn/
utils/
4 lines
utils/

Diff 39296

View Options

lib/actions/upload-actions.js

Loading...
View Options

lib/keyserver-conn/call-single-keyserver-endpoint.js

Loading...
View Options

lib/keyserver-conn/upload-blob.js

Loading...
View Options

lib/utils/upload-blob.js

Loading...