Skip to content

Add method to publish without client serialization#1010

Merged
halspang merged 2 commits into
dapr:masterfrom
halspang:rawMessage
Jan 23, 2023
Merged

Add method to publish without client serialization#1010
halspang merged 2 commits into
dapr:masterfrom
halspang:rawMessage

Conversation

@halspang
Copy link
Copy Markdown
Contributor

Description

This commit adds the ability to publish raw bytes instead of using the serialization that is built into the client.

#718

Signed-off-by: halspang halspang@microsoft.com

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #718

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

This commit adds the ability to publish raw bytes instead of using
the serialization that is built into the client.

dapr#718

Signed-off-by: halspang <halspang@microsoft.com>
@halspang halspang requested review from a team as code owners January 13, 2023 01:31
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 13, 2023

Codecov Report

Merging #1010 (af20876) into master (117c5ad) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1010      +/-   ##
==========================================
+ Coverage   69.88%   69.90%   +0.01%     
==========================================
  Files         157      157              
  Lines        5224     5227       +3     
  Branches      562      562              
==========================================
+ Hits         3651     3654       +3     
  Misses       1440     1440              
  Partials      133      133              
Flag Coverage Δ
net5 69.86% <100.00%> (+0.01%) ⬆️
net6 69.82% <100.00%> (+0.01%) ⬆️
netcoreapp3.1 69.86% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/Dapr.Client/DaprClient.cs 92.59% <ø> (ø)
src/Dapr.Client/DaprClientGrpc.cs 86.73% <100.00%> (+0.05%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Copy Markdown
Contributor

@rynowak rynowak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit: - looks straightforward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for publishing raw messages(non cloud events)

2 participants