specification icon indicating copy to clipboard operation
specification copied to clipboard

Etag support

Open aijkl opened this issue 4 years ago • 4 comments
trafficstars

I think this is a great library. Thank you. On a quick note, please consider supporting Etag.

aijkl avatar Aug 30 '21 09:08 aijkl

こんばんはおれんじみかんさん、

Thank you for the suggestion. I didn't know VRChat supported ETag, but just tested with a If-None-Match tag on a GET request and it worked. ✔️ Although it does not seem to support If-Match tags on PUT, so this can't be used for atomic update control. :x:

Can you please elaborate a little on what is the use-case for ETag support?

Foorack avatar Aug 30 '21 10:08 Foorack

I was busy with my summer vacation homework, and I'm sorry for the late reply. Japanese schools are on summer vacation until September 1st, so there is a war on August 31st. I wanted to use an Etag cache for things that aren't updated often, such as world details.

I don't know about atomic update control, but I don't want you to force it. Binary data such as images is essential, but it does not have to be text content such as Json.

I am using Google Translate, so the translation may be strange.

aijkl avatar Sep 01 '21 02:09 aijkl

File uploading also requires etags.

binn avatar Dec 30 '21 21:12 binn

We should investigate if OpenAPI supports this or if we can do something to support it at all. @Foorack, are you available to look into this?

ariesclark avatar May 25 '22 05:05 ariesclark