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

[Service Bus] Add skipParsingBodyAsJson client option to disable attempting to parse message body as Json #18630

Closed
jeremymeng opened this issue Nov 10, 2021 · 0 comments · Fixed by #18692
Closed

[Service Bus] Add skipParsingBodyAsJson client option to disable attempting to parse message body as Json #18630

jeremymeng opened this issue Nov 10, 2021 · 0 comments · Fixed by #18692
Assignees
@jeremymeng
Labels
Client This issue points to a problem in the data-plane of the library. Service Bus
Milestone
[2021] December

Comments

@jeremymeng
Copy link
Member

We've done it for Event Hubs in PR #18173. This tracks similar work for Service Bus.
@jeremymeng jeremymeng added Client This issue points to a problem in the data-plane of the library. Service Bus labels Nov 10, 2021
@jeremymeng jeremymeng added this to the [2021] December milestone Nov 10, 2021
@jeremymeng jeremymeng self-assigned this Nov 10, 2021
@jeremymeng jeremymeng mentioned this issue Nov 10, 2021
Merged
8 tasks
jeremymeng added a commit to jeremymeng/azure-sdk-for-js that referenced this issue Nov 16, 2021
@jeremymeng
[ServiceBus] Add optional boolean skipParsingBodyAsJson option
b7adad9 
to `ReceiveMessagesOptions`, `SubscribeOptions`, and
`ServiceBusSessionReceiverOptions`. This allows users to control whether the SDK
should skip parsing message body as Json object. By default the SDK will attempt
parsing message body as Json object.

While updating code, I also fixed sample unit test which is using out-dated
paths thus not finding any sample code files, and removed two `await` that are
redundant.

Resolves Azure#18630
@jeremymeng jeremymeng mentioned this issue Nov 16, 2021
Merged
jeremymeng added a commit that referenced this issue Dec 10, 2021
@jeremymeng @deyaaeldeen
[ServiceBus] Add optional boolean skipParsingBodyAsJson option (#18692
3bdb490 
)

* [ServiceBus] Add optional boolean `skipParsingBodyAsJson` option

to `ReceiveMessagesOptions`, `SubscribeOptions`, and
`ServiceBusSessionReceiverOptions`. This allows users to control whether the SDK
should skip parsing message body as Json object. By default the SDK will attempt
parsing message body as Json object.

While updating code, I also fixed sample unit test which is using out-dated
paths thus not finding any sample code files, and removed two `await` that are
redundant.

Resolves #18630

* Add optional `skipParsingBodyAsJson` boolean property

to `ServiceBusReceiverOptions` and remove those on `ReceiveMessagesOptions` and
`SubscribeOptions`

* Remove commented code that was trying to decode again

but the message body is already decoded, and _rawAmqpMessage is assigned in
`fromRheaMessage()`

* Remove double decoding

message.body has been decoded already above by `fromRheaMessage()` call.

* Update CHANGELOG and package version

* Add comments

* Update sdk/servicebus/service-bus/CHANGELOG.md

Co-authored-by: Deyaaeldeen Almahallawi <dealmaha@microsoft.com>

Co-authored-by: Deyaaeldeen Almahallawi <dealmaha@microsoft.com>
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-js that referenced this issue May 5, 2022
@moranraz @jungph808
@rpressburger @ityankel @ArcturusZhang
CodeGen from PR 18796 in Azure/azure-rest-api-specs
0dfa0fd 
dev-Sentinel-2022-05-01-preview (Azure#18796)

* Adds base for updating Microsoft.SecurityInsights from version preview/2022-04-01-preview to version 2022-05-01-preview

* Updates readme

* Updates API version in new specs and examples

* Added fileImports.json (Azure#18540)

* Added fileImports.json

* add example files

* move example files

* delete old example files

* fix get file import by id example

* fix delete file import example

* fix typos

* resolve issues

* fix date

* more issue resolving

* .

* .

* add location to delete example

* support 200

* resolve comments

* update example

* remove whitespace

* readonly

* operationId

* revert file imports (Azure#18663)

* add securityMLAnalyticsSettings to securityInsights (Azure#18679)

* add securityMLAnalyticsSettings to securityInsights

* update for operationIdNounConflictingModelNames

* update plurality

* update

* add settingsdefinintionid field for anomalySettings

* Add Anomaly to timeline items (Azure#18675)

* Add Anomaly to timeline items

* Add exmaple of anomaly

* Add x-ms-identifiers

* fix  issue
fix x-m-id issue.
fix lint

* Change enum name

* REmove timeline from inceidents

* swap enum definistion and name

* Align SKU descriptions (Azure#18874)

* Align SKU descriptions

* Remove Sku model from Settings as it's not in use and has conflict with another trype.

* Update AutomationRules.json (Azure#18630)

* Update automation rules swagger

* Update examples

* fix examples

* fix naming typo

* run prettier

* run prettier on missing file

* remove logic app arm id from required

* remove a tag with errors (Azure#18916)

Co-authored-by: moranraz <59017169+moranraz@users.noreply.github.com>
Co-authored-by: jungph808 <100548649+jungph808@users.noreply.github.com>
Co-authored-by: rpressburger <93909726+rpressburger@users.noreply.github.com>
Co-authored-by: ityankel <47292630+ityankel@users.noreply.github.com>
Co-authored-by: Dapeng Zhang <dapzhang@microsoft.com>
@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jeremymeng jeremymeng

Labels
Client This issue points to a problem in the data-plane of the library. Service Bus
Projects
None yet
Milestone
[2021] December
Development

Successfully merging a pull request may close this issue.

[ServiceBus] Add optional boolean skipParsingBodyAsJson option jeremymeng/azure-sdk-for-js
1 participant
@jeremymeng