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

[QUERY] Accessing QueueMessage.MessageText as byte array #14705

Closed
SeanFeldman opened this issue Aug 31, 2020 · 6 comments
Closed

[QUERY] Accessing QueueMessage.MessageText as byte array #14705

SeanFeldman opened this issue Aug 31, 2020 · 6 comments
Assignees
@kasobol-msft
Labels
Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. feature-request This issue requires a new behavior in the product in order be resolved. Service Attention Workflow: This issue is responsible by Azure service team. Storage Storage Service (Queues, Blobs, Files)
Milestone
Backlog

Comments

@SeanFeldman
Copy link
Contributor

SeanFeldman commented Aug 31, 2020 •
edited
Loading

Query/Question

In WindowsAzure.Storage SDK, CloudQueueMessage payload could be accessed as an array of bytes using .AsBytes.
With Azure.Storage.Queues the only option to access the payload is as text using MessageText.

How to access message payload as a byte array?

Update: The support for byte array was in SDK v11 and the change was in v12, dropping the support for byte[].

Environment:

  • Name and version of the Library package used: Azure.Storage.Blobs 12.*
@ghost ghost added needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Aug 31, 2020
@jsquire jsquire added Client This issue points to a problem in the data-plane of the library. needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team Service Attention Workflow: This issue is responsible by Azure service team. Storage Storage Service (Queues, Blobs, Files) labels Aug 31, 2020
@ghost ghost removed the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Aug 31, 2020
@ghost
Copy link

ghost commented Aug 31, 2020

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @xgithubtriage.

@jsquire
Copy link
Member

jsquire commented Aug 31, 2020

Hi Sean. Thanks for your feedback. Tagging and routing to the team best able to assist.

@SeanFeldman SeanFeldman changed the title [QUERY] QueueCloud payload as byte [QUERY] QueueCloud payload as byte array Sep 1, 2020
@tg-msft tg-msft changed the title [QUERY] QueueCloud payload as byte array [QUERY] Accessing QueueMessage.MessageText as byte array Sep 1, 2020
@tg-msft
Copy link
Member

tg-msft commented Sep 1, 2020

+@kasobol-msft who is thinking about this.

@kasobol-msft
Copy link
Contributor

@SeanFeldman V11 is automatically encoding byte[] to base64 string which is then send over the wire. Currently the equivalent in V12 would be to Base64 encode byte[] before calling SDK.
However, as @tg-msft said, we're working on handling this case in V12 SDK. This will come in one of our future releases.

@SeanFeldman
Copy link
Contributor Author

@kasobol-msft, is the plan and decision documented anywhere? I'd like to see the proposed change. Thank you.

@amishra-dev amishra-dev added feature-request This issue requires a new behavior in the product in order be resolved. and removed question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Oct 23, 2020
@jsquire jsquire added this to the Backlog milestone Oct 26, 2020
@jsquire jsquire removed the needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team label Oct 26, 2020
@kasobol-msft
Copy link
Contributor

This has been addressed in #16689 along with base64 option and available in https://www.nuget.org/packages/Azure.Storage.Queues/12.5.0

openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-net that referenced this issue Jul 1, 2021
@VinayRSoni @SubaMoorthy @joshanator
CodeGen from PR 14705 in Azure/azure-rest-api-specs
6e928ee 
Remove WorkspaceConnectionDto and use WorkspaceConnection instead for PUT request (Azure#14705)

* Add Identity to createWorkpace example

* change for getTransitivePEUsages

* update

* prettier code

* add another new endpoint

* address comments

* change for getTransitivePEUsages

* update

* prettier code

* add another new endpoint

* address comments

* revert change on transitivePE and notebookKeys

* revert change on transitivePE and notebookKeys

* add new stable version 2021-01-01

* add ListNotebookAccessToken

* add primaryUserAssignedIdentity

* fix missing properties and apis

* fix one more missing property

* 202 status for start and stop CI

* make resync keys long runnin operation in version Jan2021

* resync keys 204

* update resync keys example

* Dont throw 204 for resync

* update resync keys

* Formatting

* revert Jan2021changes and add new version

* start apr2021 version for azure ml rp

* undo remove new line in jan2021

* use systemData definition in common

* removed old files

* fix small swagger bugs

* run prettier

* remove reference to sku

* update readme

* minimum agentCount should be 0

* add createdOn and ModifiedOn to compute properties

* add LoadBalancerType to aks compute

* add isNotebookInstanceCompute to virtualMachine

* add leafDomainLabel to SslConfiguration

* add overwriteExistingDomain to SslConfiguration

* add synapseSpark as supported compute-type

* update patch compute response to 202

* remove location from example responses

* Revert "remove location from example responses"

This reverts commit 048bf2551513afae842a9c4fbb5861aeddc6c33a.

* MachineLearningCompute_Update should have a 200 response

* sys data to use local def due to naming conflicts.

* Fix enum casing

* Fix operationId PR comment.

* Change opId in apr version

* remove 20210401 for easy review

* only change api version

* merge changes for 20210401 version

* Add x-ms-discriminator-value for Synapse spark

* camelcase loadbalancer properties

* add createdOn and Modified on to examples

* address AvoidAnonymousTypes error for SynapseSpark

* Minor fixes + new property

* Remove subscription state.

* fix swagger bug

* fix example

* fix more reported problems

* remove nullable flag for tags

* make containerRegistry nullable

* address comments

* fix example

Co-authored-by: Vinay Rajmal Soni <[email protected]>
Co-authored-by: Suba <[email protected]>
Co-authored-by: [email protected] <[email protected]>
Co-authored-by: Josh Peterson <[email protected]>
Co-authored-by: Suba <[email protected]>
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@kasobol-msft kasobol-msft

Labels
Client This issue points to a problem in the data-plane of the library. customer-reported Issues that are reported by GitHub users external to the Azure organization. feature-request This issue requires a new behavior in the product in order be resolved. Service Attention Workflow: This issue is responsible by Azure service team. Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
5 participants
@jsquire @tg-msft @SeanFeldman @amishra-dev @kasobol-msft