Skip to content
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.

Sign up for GitHub

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

Jump to bottom

api: PUT /eval endpoint doesn't need blob #124

Open
seabo opened this issue Nov 27, 2022 · 0 comments
Open

api: PUT /eval endpoint doesn't need blob #124

seabo opened this issue Nov 27, 2022 · 0 comments
Assignees
@seabo
Labels
api Related to HitSave server

Comments

@seabo
Copy link
Contributor

seabo commented Nov 27, 2022 •
edited by EdAyers
Loading

Change the API for PUT /eval so that we don't need to send the associated blob in the same HTTP request.

Instead, there is a digest field on the metadata json containing the digest of the blob.
@seabo seabo added the api Related to HitSave server label Nov 27, 2022
@seabo seabo self-assigned this Nov 27, 2022
@seabo seabo added this to the v1 milestone Nov 27, 2022
@seabo seabo mentioned this issue Dec 12, 2022
Merged
EdAyers added a commit that referenced this issue Dec 12, 2022
@EdAyers
feat: client code for #124
fd27340
EdAyers added a commit that referenced this issue Dec 12, 2022
@EdAyers
feat: client code for #124
4039901
@EdAyers EdAyers removed this from the v1 milestone Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@seabo seabo

Labels
api Related to HitSave server
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@EdAyers @seabo