Skip to content

[Backport release-1.17] feat: attach SDK User-Agent#1764

Merged
javier-aliaga merged 1 commit into
release-1.17from
backport-1758-to-release-1.17
Jun 10, 2026
Merged

[Backport release-1.17] feat: attach SDK User-Agent#1764
javier-aliaga merged 1 commit into
release-1.17from
backport-1758-to-release-1.17

Conversation

@dapr-bot

@dapr-bot dapr-bot commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Backport 43cf7c6 from #1758.

…odyPublishers.json scope (#1758)

newRequestBuilder now sets the User-Agent header to the SDK version, matching
what the deprecated invokeMethod path emitted via DaprHttp. This is the only
default the SDK can supply that callers cannot, and it is useful for runtime
side triage. Other implicit invokeMethod behaviors (Content-Type default,
request-id, error mapping, trace propagation) are intentionally left to the
caller, since exposing the native HttpClient is the point of the migration.

Also document on DaprBodyPublishers.json that the helper uses the SDK's
default object serializer and is not a wrapper around a configured custom
DaprObjectSerializer. Its only contribution over a plain ofByteArray call is
guaranteeing a length-known BodyPublisher, which keeps the JDK on
Content-Length framing and sidesteps the Transfer-Encoding: chunked race
reported under load.

Signed-off-by: Javier Aliaga <javier@diagrid.io>
(cherry picked from commit 43cf7c6)
@dapr-bot dapr-bot requested review from a team as code owners June 9, 2026 13:45
@codecov

codecov Bot commented Jun 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.98%. Comparing base (c02ab76) to head (7cd1324).

Additional details and impacted files
@@                Coverage Diff                 @@
##             release-1.17    #1764      +/-   ##
==================================================
- Coverage           72.98%   72.98%   -0.01%     
- Complexity           2275     2276       +1     
==================================================
  Files                 243      243              
  Lines                7437     7439       +2     
  Branches              739      739              
==================================================
+ Hits                 5428     5429       +1     
- Misses               1649     1651       +2     
+ Partials              360      359       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@javier-aliaga javier-aliaga merged commit 2d6af6e into release-1.17 Jun 10, 2026
36 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants