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

Content-Length is empty on HEAD #209

Closed
gerhardj-b opened this issue Oct 10, 2022 · 2 comments · Fixed by #219
Closed

Content-Length is empty on HEAD #209

gerhardj-b opened this issue Oct 10, 2022 · 2 comments · Fixed by #219
Assignees
@jakobvogel
Labels
in progress 🛫

Comments

@gerhardj-b
Copy link

This is not according to the S3 API as well as HTTP standards.

My client relies on the content length returned by the HEAD request. Is it therefore possible to change this, so that the content-length is also sent on such requests?
@jakobvogel jakobvogel self-assigned this Feb 16, 2023
@jakobvogel
Copy link
Member

Many thanks for reporting this @gerhardj-b ☺️ … Can you please be more precise? Which operations exactly expect the content-length and we fail to provide it?

@jakobvogel
Copy link
Member

I have found one such condition on PUTing an object. Do you refer to this?

jakobvogel added a commit that referenced this issue Feb 23, 2023
@jakobvogel
Reports content-length of streaming data 📐
289a46a 
This avoids warnings by the AWS client software.

#209
SIRI-721

Signed-off-by: Jakob Vogel <jvo@scireum.de>
@jakobvogel jakobvogel mentioned this issue Feb 23, 2023
Merged
jakobvogel added a commit that referenced this issue Feb 23, 2023
@jakobvogel
Moves helper method up 👆
0e9390d 
Signed-off-by: Jakob Vogel <jvo@scireum.de>

#209
jakobvogel added a commit that referenced this issue Feb 24, 2023
@jakobvogel
Uses standard charsets 👨‍⚖️
3b5ad70 
Signed-off-by: Jakob Vogel <jvo@scireum.de>

#209
jakobvogel added a commit that referenced this issue Feb 24, 2023
@jakobvogel
Adds helper for creating publicly accessible buckets while testing 🔬
a017e98 
#209
SIRI-721

Signed-off-by: Jakob Vogel <jvo@scireum.de>
jakobvogel added a commit that referenced this issue Feb 24, 2023
@jakobvogel
Adds test case checking the content-length when responding to HEAD 🦁
f45baa6 
#209
SIRI-721

Signed-off-by: Jakob Vogel <jvo@scireum.de>
jakobvogel added a commit that referenced this issue Feb 24, 2023
@jakobvogel
Upgrades Sirius 🪐
86859b9 
#209
SIRI-721

Signed-off-by: Jakob Vogel <jvo@scireum.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jakobvogel jakobvogel

Labels
in progress 🛫
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix: Reporting of Content Length scireum/s3ninja
2 participants
@jakobvogel @gerhardj-b