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.

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

Fix for #19724, upload_blob uploads 0 bytes for socket stream. #19725

Merged
merged 1 commit into from
Sep 3, 2021

Conversation

jessewmc
Copy link
Contributor

@jessewmc jessewmc commented Jul 8, 2021

See issue: #19724

Check fd is reg file or symlink in get_length before using st_size.

st_size is not meaningful for non-regular files (or symlinks).
See man (2) stat.

st_size is not meaningful for non-regular files (or symlinks).
See man (2) stat.
@ghost ghost added Storage Storage Service (Queues, Blobs, Files) customer-reported Issues that are reported by GitHub users external to the Azure organization. labels Jul 8, 2021
@ghost
Copy link

ghost commented Jul 8, 2021

Thank you for your contribution jessewmc! We will review the pull request and get back to you soon.

@xiafu-msft xiafu-msft merged commit d79015f into Azure:main Sep 3, 2021
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request Sep 7, 2021
…into switch_to_protocol

* 'main' of https://github.com/Azure/azure-sdk-for-python: (53 commits)
  Smoke test package verification (Azure#20547)
  Update CHANGELOG.md (Azure#20569)
  updating codeowners (Azure#20570)
  Simplify recorded OnBehalfOfCredential tests (Azure#20568)
  Search: update release date (Azure#20564)
  [Exporter] Support redirect response in exporter (Azure#20489)
  [AutoRelease] t2-synapse-2021-09-06-18774 (Azure#20552)
  [AutoRelease] t2-logz-2021-08-26-01773 (Azure#20426)
  [AutoRelease] t2-relay-2021-09-03-32777 (Azure#20530)
  [AutoRelease] t2-resource-2021-09-03-61345 (Azure#20527)
  [AutoRelease] t2-servicefabric-2021-09-02-41879 (Azure#20512)
  [Keyvault] Remove exception message parsing from samples in keys, certificates and secrets (Azure#20540)
  [ACR] Update cloud configuration API (Azure#20464)
  Get rid of generated code (Azure#20536)
  Search: Skill Versions (Azure#20431)
  Check fd is reg file or symlink in get_length before using st_size. (Azure#19725)
  Add OnBehalfOfCredential (Azure#20451)
  Fix incorrect parsing of message from Exception (Azure#20534)
  [Release sdk status] swagger repo default branch changes to `main` (Azure#20529)
  [AutoRelease] t2-rdbms-2021-09-02-91864 (Azure#20513)
  ...
iscai-msft pushed a commit that referenced this pull request Sep 29, 2021
…19725)

st_size is not meaningful for non-regular files (or symlinks).
See man (2) stat.
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-python that referenced this pull request Sep 19, 2022
[Microsoft.StorageSync] New API version 2022-06-01 (Azure#20227)

* Adds base for updating Microsoft.StorageSync from version stable/2020-09-01 to version 2022-06-01

* Updates readme

* Updates API version in new specs and examples

* Swagger API Spec Update for adding Low Disk Mode to Storage Sync (Azure#19725)

* Added Low Disk Mode property

* Updated API Spec for introducing Low Disk Mode

* Updated Examples for API Spec for Low Disk Mode

* Updated Examples for Cloud Tiering Status (Azure#19885)

* [Microsoft.StorageSync] Add AFS file share metadata certificate public keys API to version 2022-06-01 (S360 compliance) (Azure#20517)

* Fix readme.md file so that tag information includes full API version 2022-06-01

* Add afs share metadata certificate public keys API

https://portal.azure-devex-tools.com/amekpis/completeness/detail?errorId=DBF8D800-EB0B-48E0-B665-BC91CB1A17B9

* Fix casing of afssharemetadatacertificatepublickeys

* Temporarily rename examples file to fix casing issue.

* Fix casing of new example file

Co-authored-by: ankushbindlish2 <34896519+ankushbindlish2@users.noreply.github.com>
Co-authored-by: Juan Carlos Juarez <106784687+jcjuarez-msft@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants