Skip to content

Disable Protocol V4 in invoker - #5259

Open
slinkydeveloper wants to merge 2 commits into
restatedev:mainfrom
slinkydeveloper:disable-protocol-v4-invoker
Open

Disable Protocol V4 in invoker#5259
slinkydeveloper wants to merge 2 commits into
restatedev:mainfrom
slinkydeveloper:disable-protocol-v4-invoker

Conversation

@slinkydeveloper

Copy link
Copy Markdown
Contributor

This will auto fail requests to the invoker that use protocol <= v3. People can restart invocations using restart as new, if they need so.

As followup is probably worth re-merging back service prtocol runner with invocation task, as they're basically the same chunk of logic.

Part of #3184

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@muhamadazmy muhamadazmy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚢 after fixing CI

@slinkydeveloper
slinkydeveloper force-pushed the disable-protocol-v4-invoker branch from 72ccfa2 to 801c03f Compare August 28, 2026 14:03
@slinkydeveloper
slinkydeveloper force-pushed the disable-protocol-v4-invoker branch from 801c03f to 162111a Compare August 28, 2026 14:36
@slinkydeveloper slinkydeveloper added the release-blocker Blocker for the next release label Sep 2, 2026
@slinkydeveloper slinkydeveloper added this to the 1.8 milestone Sep 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-blocker Blocker for the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants