Skip to content

proxy-request.expect: absent in PV replays#13054

Merged
bneradt merged 1 commit intoapache:masterfrom
bneradt:codex/pv-expect-absent-replays
Apr 3, 2026
Merged

proxy-request.expect: absent in PV replays#13054
bneradt merged 1 commit intoapache:masterfrom
bneradt:codex/pv-expect-absent-replays

Conversation

@bneradt
Copy link
Copy Markdown
Contributor

@bneradt bneradt commented Apr 2, 2026

Proxy Verifier v3.1.2 can now verify that ATS does not send an upstream
request to the origin. Update replay transactions that serve from cache
or otherwise respond locally to use proxy-request.expect: absent.

This makes the no-origin cases explicit across the autests and turns the
old trap server responses into a direct expectation. It also fixes one
stale comment in the HEAD/GET cache replay so the test text matches the
origin behavior.

@bneradt bneradt added this to the 11.0.0 milestone Apr 2, 2026
@bneradt bneradt self-assigned this Apr 2, 2026
@bneradt bneradt force-pushed the codex/pv-expect-absent-replays branch from 733adc6 to 5ec4d25 Compare April 2, 2026 20:08
@bneradt bneradt changed the title Assert absent proxy requests in replays proxy-request.expect: absent in PV replays Apr 2, 2026
Proxy Verifier v3.1.2 can now assert that ATS does not send an
upstream request. Update replay transactions that serve from cache or
otherwise respond locally to use proxy-request.expect: absent.

This makes the no-origin cases explicit across the gold tests and
turns the old trap server responses into a direct expectation. Leave
the cache-control-basic miss transaction alone because it must still go
upstream to prime the cache, and fix one stale HEAD/GET cache comment
so the replay text matches the origin behavior.
@bneradt bneradt force-pushed the codex/pv-expect-absent-replays branch from 9e1110e to 8ec503d Compare April 2, 2026 22:23
Copy link
Copy Markdown
Contributor

@masaori335 masaori335 left a comment

Choose a reason for hiding this comment

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

Nice! This clarify behavior of ATS.

@bneradt bneradt merged commit bf36ea7 into apache:master Apr 3, 2026
15 checks passed
@bneradt bneradt deleted the codex/pv-expect-absent-replays branch April 3, 2026 17:20
@github-project-automation github-project-automation bot moved this to For v10.2.0 in ATS v10.2.x Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: For v10.2.0

Development

Successfully merging this pull request may close these issues.

2 participants