Page MenuHomePhabricator
Differential D7620

[lib][keyserver] Expose upload_media_metadata endpoint
ClosedPublic
Actions

Authored by bartek on Apr 25 2023, 11:28 AM.
Tags
None
Referenced Files
F1690460: D7620.diff
Wed, May 1, 9:43 PM
Unknown Object (File)
Tue, Apr 23, 9:42 AM
Unknown Object (File)
Wed, Apr 3, 8:22 PM
Unknown Object (File)
Wed, Apr 3, 8:22 PM
Unknown Object (File)
Wed, Apr 3, 8:22 PM
Unknown Object (File)
Wed, Apr 3, 8:22 PM
Unknown Object (File)
Wed, Apr 3, 8:21 PM
Unknown Object (File)
Wed, Apr 3, 8:14 PM
View All 68 Files
Subscribers
tomek

Details

Reviewers
ashoat
atul
marcin
inka
Commits
rCOMMfc743d8532ea: [lib][keyserver] Expose upload_media_metadata endpoint
Summary

This diff exposes the upload_media_metadata endpoint.

Depends on D7619

Test Plan

Created useServerCall hook and called this endpoint. Verified that a new record is added to the uploads table.
The endpoint response format is the same as the upload_multimedia endpoint.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek created this revision.Apr 25 2023, 11:28 AM
bartek held this revision as a draft.
Herald added a subscriber: tomek. · View Herald TranscriptApr 25 2023, 11:28 AM
bartek published this revision for review.Apr 25 2023, 11:46 AM
Harbormaster completed remote builds in B18754: Diff 25694.Apr 25 2023, 11:48 AM
ashoat accepted this revision.Apr 25 2023, 12:48 PM
ashoat added inline comments.
lib/actions/upload-actions.js
28 ↗(On Diff #25694)

Since we directly return the result from the server here, it would be good to make sure that the code that consumes this will not break if the server starts including some new additional key

This revision is now accepted and ready to land.Apr 25 2023, 12:48 PM
bartek updated this revision to Diff 25809.Apr 27 2023, 3:47 AM

Rebase. Opted out of direct return, for similiar reasons as discussed in D7522

Harbormaster completed remote builds in B18840: Diff 25809.Apr 27 2023, 4:07 AM
bartek added a child revision: D7649: [native] Add function to upload media to blob service.Apr 27 2023, 4:31 AM
bartek added a child revision: D7689: [web] Add function to upload media to Blob service.Apr 29 2023, 9:20 AM
bartek updated this revision to Diff 26191.May 8 2023, 4:54 AM

Rebase before landing

Harbormaster completed remote builds in B19131: Diff 26191.May 8 2023, 5:12 AM
Closed by commit rCOMMfc743d8532ea: [lib][keyserver] Expose upload_media_metadata endpoint (authored by bartek). · Explain WhyMay 8 2023, 5:19 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMMfc743d8532ea: [lib][keyserver] Expose upload_media_metadata endpoint.

Revision Contents
Changeset List

PathSize
keyserver/
src/
9 lines
lib/
actions/
19 lines
types/
1 line

Diff 26191

View Options

keyserver/src/endpoints.js

Loading...
View Options

lib/actions/upload-actions.js

Loading...
View Options

lib/types/endpoints.js

Loading...