Skip to content

[1.18] Updates resiliency of streaming HTTP Service Invocation#5069

Open
JoshVanL wants to merge 2 commits intodapr:v1.18from
JoshVanL:sa-http-streaming-1.18
Open

[1.18] Updates resiliency of streaming HTTP Service Invocation#5069
JoshVanL wants to merge 2 commits intodapr:v1.18from
JoshVanL:sa-http-streaming-1.18

Conversation

@JoshVanL
Copy link
Contributor

Based on this PR dapr/dapr#9627

Based on this PR dapr/dapr#9627

Signed-off-by: joshvanl <me@joshvanl.dev>
@JoshVanL JoshVanL requested review from a team as code owners March 11, 2026 16:21
Copilot AI review requested due to automatic review settings March 11, 2026 16:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Dapr 1.18 documentation to clarify how resiliency behaves for streaming HTTP service invocation, specifically that retry policies are bypassed when the request body can’t be replayed.

Changes:

  • Add notes to resiliency targets and retry policy docs explaining retries are skipped for streaming HTTP requests.
  • Add a note + a new “Streaming requests and resiliency” subsection to the service invocation overview describing detection and behavior.
  • Minor markdown formatting cleanup in the service invocation streaming section.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
daprdocs/content/en/operations/resiliency/targets.md Adds a note on streaming HTTP invocation bypassing retries and linking to more details.
daprdocs/content/en/operations/resiliency/policies/retries/retries-overview.md Adds a note explaining why streaming HTTP requests can’t be retried.
daprdocs/content/en/developing-applications/building-blocks/service-invocation/service-invocation-overview.md Adds an alert + a dedicated subsection documenting streaming detection and resiliency behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Josh van Leeuwen <me@joshvanl.dev>
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.

2 participants