Skip to content

chore(test): replace netcat with Python HTTP server in network tests#1140

Merged
dobrac merged 1 commit intomainfrom
fix/mask-request-host-test
Feb 17, 2026
Merged

chore(test): replace netcat with Python HTTP server in network tests#1140
dobrac merged 1 commit intomainfrom
fix/mask-request-host-test

Conversation

@dobrac
Copy link
Copy Markdown
Contributor

@dobrac dobrac commented Feb 17, 2026

Note

Low Risk
Test-only changes that swap the header-capture mechanism; no production code or security-sensitive logic is modified.

Overview
Updates the JS SDK and Python SDK network tests for maskRequestHost to stop installing/using netcat to capture inbound headers.

The tests now launch a lightweight Python http.server handler that writes received request headers to /tmp/headers.txt, then read that file to assert the masked Host value; related waits/timeouts and command options were adjusted to match the new approach.

Written by Cursor Bugbot for commit a6992a6. This will update automatically on new commits. Configure here.

@dobrac dobrac added the bug Something isn't working label Feb 17, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Feb 17, 2026

⚠️ No Changeset found

Latest commit: a6992a6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dobrac dobrac changed the title test: replace netcat with Python HTTP server in network tests chore(test): replace netcat with Python HTTP server in network tests Feb 17, 2026
@dobrac dobrac merged commit 3033d25 into main Feb 17, 2026
20 of 22 checks passed
@dobrac dobrac deleted the fix/mask-request-host-test branch February 17, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants