Skip to content

Add L7 proxy request resolution path#342

Merged
kacy merged 1 commit intomainfrom
phase6-proxy-execution-path
Mar 27, 2026
Merged

Add L7 proxy request resolution path#342
kacy merged 1 commit intomainfrom
phase6-proxy-execution-path

Conversation

@kacy
Copy link
Copy Markdown
Owner

@kacy kacy commented Mar 27, 2026

Summary

  • add a pure L7 request-resolution path for the HTTP proxy: parse request, match route, select eligible upstream, and return a forward plan
  • return explicit 400 for malformed requests or missing Host, 404 for unmatched routes, and 503 when no eligible upstream exists
  • extend runtime and upstream helpers so reverse_proxy can resolve routes and backend endpoints from canonical service state

Commit

  • e0c073c Add L7 proxy request resolution path

Validation

  • timeout 45 env YOQ_SKIP_SLOW_TESTS=1 ZIG_GLOBAL_CACHE_DIR=.zig-global-cache ZIG_LOCAL_CACHE_DIR=.zig-local-cache zig build test -Doptimize=ReleaseSafe -- --test-filter "handleRequest returns forward plan for a routable request"
  • timeout 45 env YOQ_SKIP_SLOW_TESTS=1 ZIG_GLOBAL_CACHE_DIR=.zig-global-cache ZIG_LOCAL_CACHE_DIR=.zig-local-cache zig build test -Doptimize=ReleaseSafe -- --test-filter "handleRequest returns service unavailable when route has no eligible upstream"

Both bounded test runs exited on timeout without emitted failures.

@kacy kacy merged commit cdf4487 into main Mar 27, 2026
6 of 7 checks passed
@kacy kacy deleted the phase6-proxy-execution-path branch March 27, 2026 16:41
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.

1 participant