chore: scrub adult-site domain from smoke-test workflows (ENG-1347) - #100
chore: scrub adult-site domain from smoke-test workflows (ENG-1347)#100jadoonf wants to merge 2 commits into
Conversation
Replace the xvideos.com curl probes in the PR-comment smoke tests with oastify.com, which sits on the embedded malicious-domains blocklist, so the steps still exercise known-bad egress detection without an adult-site domain in the public repo before Marketplace visibility.
|
See what ran — every outbound connection and its process lineage.
💡 Reading this reviewEach job fold connects recorded process lineage to recorded outbound destinations. Bold lineage is attributed to a GitHub step below Runner.Worker
└─ bash
└─ curl
└─ → example.com
1 job recorded on this commit
|
|
See what ran — every process your jobs executed, and where they connected
3 jobs across 2 workflows recorded on this commit
|
Summary
Removes the last adult-site domain from the repo before the Marketplace listing goes public (Linear ENG-1347).
curl https://xvideos.comprobe steps in the PR-comment smoke-test workflows (.github/workflows/pr-comment-smoke-test-a-and-b.yaml×2 jobs,pr-comment-smoke-test-c.yaml×1).oastify.com(an Interactsh OOB-testing domain already on the control-plane's embeddedmalicious-domainsblocklist), so the step still exercises the known-bad-egress detection path — the test semantics ("resolvable domain that triggers a detection") are preserved, unlike a documentation-reserved domain that would never resolve or connect.test/fixtures/profiles/*.json, goldens) and README/docs were already clean; the runtime-review-testbed repo scans clean too.Test plan
rg -i --hiddenfor the adult-domain pattern list returns zero hits repo-wideworkflow_dispatch/ label-gated — nextrun-smoke-testsrun should show theoastify.comprobe flagged as beforeLinear: ENG-1347