Skip to content

Resolve critical failure in high-integrity-mode with -MOVED response#3049

Merged
mgravell merged 3 commits into
mainfrom
marc/3048
Apr 2, 2026
Merged

Resolve critical failure in high-integrity-mode with -MOVED response#3049
mgravell merged 3 commits into
mainfrom
marc/3048

Conversation

@mgravell

@mgravell mgravell commented Apr 2, 2026

Copy link
Copy Markdown
Collaborator

Fix #3048 (see there for context)

@mgravell mgravell requested a review from philon-msft April 2, 2026 13:02
@mgravell

mgravell commented Apr 2, 2026

Copy link
Copy Markdown
Collaborator Author

thanks for prompt eyeballs @philon-msft

@mgravell mgravell merged commit e43dd9b into main Apr 2, 2026
8 checks passed
@mgravell mgravell deleted the marc/3048 branch April 2, 2026 13:22
@mgravell mgravell requested a review from NickCraver April 2, 2026 13:22
mgravell added a commit that referenced this pull request Apr 2, 2026
mgravell added a commit that referenced this pull request May 14, 2026
mgravell added a commit that referenced this pull request Jun 22, 2026
* WIP v3 (#3035)

* WIP v3 merge attempt

* fix refs/externs

* nits

* ascii hash cleanup

* remove unused local function

* v3 remove pipelines.sockets.unofficial (#3042)

* remove Pipelines.Sockets.Unofficial

* wip

* remove the last traces of Pipelines.Sockets.Unofficial

* fix deps

* WIP: deps

* fix build deps

* rev GitHubActionsTestLogger

* - fix delegate [UnsafeAccessor] usage
- implement Mono pattern

* merge (incomplete)

* merge

* missed a sync-context await

* fix missing configureawait usage

* missing VT consumption

* checking CI fail

* versioning

* don't test WriteMode.Sync/Async yet

* failing test for #3048

* upstream #3049 into v3

* release notes

* fix broken config test

* re-add channel handling (merge)

* optimize pub/sub check for server-defined channels

* prefer WriteBulkString("..."u8) => WriteRaw(RESPu8)

* optimize CommandMap by pre-generating all the RESP chunks

* Move CI to 8.8 rc1 (#3079)

* update CI to 8.8 rc1

* and the dockerfile

* Draft support for 8.8 arrays (#3076)

* draft support for 8.8 arrays

* - mark API as experimental
- add keyspace notification tests
- tidying
- docs

* clarify how last-items interacts with ring buffers

* fix CI netfx compilation

* use ValuePairInterleavedProcessorBase to ensure that jagged vs flat doesn't impact us (RESP2 vs RESP3)

* make life even easier

* add Array to signature prefix list

* stabilize hotkeys CI

* fix last-minute ARGREP result change

* CI: graceful skip if DEBUG not available

* Alternative AwaitableMutex implementation for netfx (#3081)

* implement AwaitableMutex for netfx

* optimize successful sync take path

* words

* optimize async path for immediate acquisition

* re-implement TCP keep-alive

* default RESP3 everywhere (#3084)

* default RESP3 everywhere

* Retain <v6 RESP2 logic; default to v6

* move ship file entries

* improve logging for ReadNotificationAsync

* stabilize CI for RetryPolicyFailureCases

* merge

* broken merge (unshipped.txt)

* add all the files

* build fix

* merge drift

* CI: allow more time for Echo

* use Task machinery for teardown test

* Drag benchmark from respite branch

* typo

* "classic"

* BufferedStreamWriter threading tests and fixes

* more clarifications on BufferedStreamWriter

* change default mode

* investigating parse bug (Garnet SPOP?)

* categorise all simulate-connection-failure tests; start working through failures

* more simulate-failure tests

* update `aoc` test expectation (fast-fail assert in test connect)

* optimize ConfigurationOptions fields

* Update CI workflow to include v3 branch for packaging

* Fix missing Me() key usage

* be explicit about test TFMs

* merge the sync and async write implementations into a shared implementation that can transition from sync to async if needed

* 3.0 release

* counting is hard
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.

Error with high-integrity mode overlapping a MOVED response

2 participants