Skip to content

DM-54789-v30: Drop host/port/scheme from VCR match_on so cassettes replay anywhere - #200

Merged
mwittgen merged 1 commit into
v30.0.xfrom
tickets/DM-54789-v30
Nov 2, 2026
Merged

mwittgen merged 1 commit into
v30.0.xfrom
tickets/DM-54789-v30

Conversation

@mwittgen

@mwittgen mwittgen commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator

The cassettes were recorded with HTTPS_PROXY active at USDF, baking sdfproxy.sdf.slac.stanford.edu:3128 into every URI. Outside USDF (and seemingly now also at USDF, after some library or NO_PROXY drift) the host/port matchers reject every replay. The recorded paths uniquely identify each service, so matching on method/path/query/body is enough and makes replay independent of proxy state.

The cassettes were recorded with HTTPS_PROXY active at USDF, baking
sdfproxy.sdf.slac.stanford.edu:3128 into every URI. Outside USDF (and
seemingly now also at USDF, after some library or NO_PROXY drift) the
host/port matchers reject every replay. The recorded paths uniquely
identify each service, so matching on method/path/query/body is enough
and makes replay independent of proxy state.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mwittgen
mwittgen merged commit 665757e into v30.0.x Nov 2, 2026
9 of 10 checks passed
@mfisherlevine

Copy link
Copy Markdown
Contributor

Is there a reason we shouldn't merge this to main as well? This seems like a change that's likely not specific to this backport, right?

@mwittgen

mwittgen commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator Author

This is a commit cherry-picked from the main branch
#189

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.

2 participants