Set `Content-Length` for s3 requests #176

New issue

Open

opened 2024-11-01 12:24:46 +00:00 by fyrchik · 2 comments

fyrchik commented

2024-11-01 12:24:46 +00:00

Owner

We have size, may send it as well.

_, err := c.cli.PutObject(c.vu.Context(), &s3.PutObjectInput{

This may or may not be an option. Likely not, but it would be nice to measure the difference in results.

FYI @dkirillov @alexvanin

We have size, may send it as well. https://git.frostfs.info/TrueCloudLab/xk6-frostfs/src/commit/f5df03c718b61cf0d994c7c35fe0736623fd10b0/internal/s3/client.go#L58 This may or may not be an option. Likely not, but it would be nice to measure the difference in results. FYI @dkirillov @alexvanin

dkirillov commented

2024-11-01 14:41:15 +00:00

Member

If you will send correct size to s3-gw that will be the same as missing content-length. So I expect no difference

fyrchik commented

2024-11-02 06:08:21 +00:00

Author

Owner

size := h.getPutPayloadSize(r)

I was under the impression that this number finds it's way into PayloadSize field of the native object.
Or no?

https://git.frostfs.info/TrueCloudLab/frostfs-s3-gw/src/commit/e35b582fe25a97c018367ef9244dfe93d4500ad9/api/handler/put.go#L246 I was under the impression that this number finds it's way into `PayloadSize` field of the native object. Or no?