-
Notifications
You must be signed in to change notification settings - Fork 841
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
[Storage][pageblob] Page blob implementation is broken, does not set required headers #22474
Comments
@tanyasethi-msft could you please take a look at this? Feel free to ping me internally. In case the type of Storage account plays a role, here are the details:
|
Hi @berndverst , Thanks for reporting this issue! I will make changes in the documentation. |
@tanyasethi-msft could you please provide a complete example that reads an arbitrary file or buffer and uploads the entire thing? That's really the sample everyone is missing. FWIW, I actually wrote other code to change offset and count values.. and everytime it would complain about other missing headers or incorrect header values. A complete example is truly needed. For example some unknowns here: We really need some clearer samples, and ideally there would be some convenience methods added to the SDK that can upload an entire file or buffer as pages using some sane defaults. |
Bug Report
The
pageblob
module (submodule ofazblob
) does not set required headers when attempting to upload a page blob toPREMIUM
storageThe proxy test which replays existing recordings works fine (run via
go test -v --count=1
) but running a test against real Azure resources does not work.I set the Storage Account environment variable and made sure I have
Azure Storage Data Owner
permission. Authentication happened successfully via the DefaultAzureCredential which used my AzureCLICredential.azure-sdk-for-go/sdk/storage/azblob/pageblob/examples_test.go
Lines 31 to 112 in 4769244
SDK Versions:
go version
go version go1.22.0 darwin/arm64
I compiled the sample (by renaming the function to main and changing the package to main, and changing the container name to something I already had) to a binary
m
and then ran it like so:Here are the full logs with Azure Go SDK debug logs enabled:
The text was updated successfully, but these errors were encountered: