Skip to content

Backport read-only mode to 0.2.x#252

Merged
em3s merged 3 commits into0.2.xfrom
feature/0.2.x-read-only
Apr 21, 2026
Merged

Backport read-only mode to 0.2.x#252
em3s merged 3 commits into0.2.xfrom
feature/0.2.x-read-only

Conversation

@eazyhozy
Copy link
Copy Markdown
Member

Summary

Backport read-only mode from `main` to the `0.2.x` release branch, so it can be included in a v0.2.1 patch release.

Changes

Note: PR #236 (Fix ReadOnlyRequestFilterTest for renamed edges/cache → edges/seek endpoint) is intentionally not included — the `edges/cache → edges/seek` rename has not been applied to 0.2.x, so applying #236 would break the test suite here.

How to Test

./gradlew :server:test --tests "*ReadOnly*" --tests "*StartUp*"
./gradlew build

eazyhozy and others added 3 commits April 21, 2026 15:11
Co-authored-by: Minseok Kim <mskim@apache.org>
(cherry picked from commit d936f56)
Co-authored-by: Minseok Kim <mskim@apache.org>
(cherry picked from commit f0a84e8)
…OINTS

On 0.2.x, POST /graph/v2/query still exists (removed in main via #228).
The filter's readSuffixes already includes /query, so this endpoint is
correctly treated as read-only. Register it in READ_ENDPOINTS so the
exhaustive-coverage test passes on this branch.
@eazyhozy eazyhozy marked this pull request as ready for review April 21, 2026 06:18
@eazyhozy eazyhozy requested a review from em3s as a code owner April 21, 2026 06:18
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 21, 2026
@eazyhozy eazyhozy self-assigned this Apr 21, 2026
@eazyhozy
Copy link
Copy Markdown
Member Author

eazyhozy commented Apr 21, 2026

@em3s

CI was not triggered because the base branch is 0.2.x instead of main. Verified locally: all 269 server tests pass with no failures.

./gradlew :server:test --rerun-tasks
→ 269 tests completed, 0 failures, 0 errors — BUILD SUCCESSFUL

@em3s em3s added this to the v0.2.1 milestone Apr 21, 2026
@em3s em3s removed their request for review April 21, 2026 08:10
@em3s em3s merged commit 697e79d into 0.2.x Apr 21, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants