-
Notifications
You must be signed in to change notification settings - Fork 2k
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.
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
[BUG] Azure Blob sdk v12 preview 3 doesn't include ETag in BlobDownloadResponse #5623
Comments
@andreaturli thank you for reporting this issue in the September preview of Azure Storage. @sima-zhu could you please investigate? /cc @amishra-dev |
Thanks @joshfree |
thanks @josefree for taking the time to have a look. Let me know if you need more info to investigate it |
Taking a look |
@andreaturli Thanks for reporting this issue! Can I get more details or code snippet on the issue? Based on the rest API doc from service.
We have several download tests and I did not run into issue for ETag. I would be great if you can provide a simple code snippet to test. |
Hi @andreaturli Do you still have the issue? If this is still a question, please leave your comments. Otherwise, I will close the issue tomorrow. Thanks! |
hi @sima-zhu I'll test it again shortly, I'll report here |
Thanks |
I still see the issue
and those are the logs from Azurite container
so the |
Can I test it on my end? |
@andreaturli BlobURL seems to the use previous version (prior to v12). |
Indeed I'm using v11 |
Can you switch to v12 to test? Or we can move to new issue regarding v11. |
Using v12 (both preview2 and preview4) gives me the same problem Using az cli I can do the following
and this is the log from Azurite
From java sdk
although azurerite server gives me the following
Notice I've tried with BlockBlobClient blockBlobClient = containerClient.getBlobClient(blobName).asBlockBlobClient();
ByteArrayOutputStream result = new ByteArrayOutputStream();
blockBlobClient.download(result); |
Looking |
I am trying to reproduce the issue but no lucky to failed at the same error. Propose a change to team. Will post here if any updates. |
What the service return to me is using ETag based on your code snippet above.
|
Could you meanwhile share the java snippet you are using to test from your end? thanks! |
The same code snippet you attached above.
|
Added new Json feature property to model class in PR #5841
We can handle over the response header name in a case-insensitive way. |
The annotation does not work. Will go with other approach |
@jianghaolu like we discussed may be the safe approach is to introduce |
PR #5844 submitted the suggested change above. |
The fix 3e03bf2 has been merged in. |
Thanks for working with Microsoft on GitHub! Tell us how you feel about your experience using the reactions on this comment. |
Describe the bug
BlobAsyncClient
doesn't parse theetag
header properly, asBlobDownloadHeaders#ETag
is using the wrong value for the@JsonProperty(value = "ETag")
instead of@JsonProperty(value = "etag")
Exception or Stack Trace
see Azure/Azurite#217 (comment)
To Reproduce
Azure/Azurite#217 (comment)
Code Snippet
Add the code snippet that causes the issue.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):
v12.0.0-preview3
Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report
The text was updated successfully, but these errors were encountered: