Allow statusCode and statusMessage to be changed on IncomingMessage

[Electroid]

What does this PR do?

ngrok is changing the statusCode and statusMessage on incoming requests, which fixes the following issue:

1212 |         this[kStopReading] = true;
1213 |         if (!(error instanceof RequestError)) {
1214 |             error = new RequestError(error.message, error, this);
1215 |         }
1216 |         const typedError = error;
1217 |         const { response } = typedError;
                    ^
RequestError: Attempted to assign to readonly property.
 code: "ERR_GOT_REQUEST_ERROR"

Unfortunately, ngrok still doesn't work. (it hangs instead of error) But I figured we should get this fix out anyway.

• Documentation or TypeScript types (it's okay to leave the rest blank in this case)
• Code changes

How did you verify your code works?

There are tests.

[github-actions]

❌ @autofix-ci[bot] 4 files with test failures on bun-darwin-aarch64:

• test/cli/test/bun-test.test.ts 1 failing
• test/integration/next/default-pages-dir/test/dev-server.test.ts 2 failing
• test/js/bun/util/stringWidth.test.ts 2 failing
• test/js/third_party/webpack/webpack.test.ts 1 failing

View test output

^{#f1509ca2274b1fe93c15fd76eccf6c48b676904e}

[github-actions]

❌ @autofix-ci[bot] 5 files with test failures on linux-x64-baseline:

• test/cli/test/bun-test.test.ts 1 failing
• test/integration/next/default-pages-dir/test/dev-server.test.ts 1 failing
• test/integration/next/default-pages-dir/test/next-build.test.ts 1 failing
• test/js/bun/util/stringWidth.test.ts 2 failing
• test/js/third_party/webpack/webpack.test.ts 1 failing

View test output

^{#f1509ca2274b1fe93c15fd76eccf6c48b676904e}

[github-actions]

❌ @autofix-ci[bot] 9 files with test failures on linux-x64:

• test/cli/test/bun-test.test.ts 1 failing
• test/integration/next/default-pages-dir/test/dev-server.test.ts 2 failing
• test/integration/next/default-pages-dir/test/next-build.test.ts 1 failing
• test/js/bun/util/stringWidth.test.ts 2 failing
• test/js/node/http2/node-http2.test.js 1 failing
• test/js/third_party/webpack/webpack.test.ts 1 failing
• test/js/web/fetch/fetch.test.ts 1 failing
• test/js/web/fetch/fetch.tls.test.ts 1 failing
• test/js/workerd/html-rewriter.test.js 1 failing

View test output

^{#f1509ca2274b1fe93c15fd76eccf6c48b676904e}

[github-actions]

❌ @autofix-ci[bot] 8 files with test failures on bun-darwin-x64:

• test/cli/test/bun-test.test.ts 2 failing
• test/integration/next/default-pages-dir/test/dev-server.test.ts 3 failing
• test/js/bun/http/bun-server.test.ts 1 failing
• test/js/bun/util/stringWidth.test.ts 2 failing
• test/js/bun/util/which.test.ts 1 failing
• test/js/node/fs/fs.test.ts 1 failing
• test/js/node/watch/fs.watchFile.test.ts 1 failing
• test/js/third_party/webpack/webpack.test.ts 1 failing

View test output

^{#f1509ca2274b1fe93c15fd76eccf6c48b676904e}

[jdalton]

For context:

Node http2 uses a symbol called 'state':
https://github.com/nodejs/node/blob/main/lib/internal/http2/compat.js#L71

Related to IncomingMessage however Node uses a data property (not accessor):
https://github.com/nodejs/node/blob/main/lib/_http_incoming.js#L88

[Jarred-Sumner]

yeah, we should just set the property on the object and not use a getter/setter

[github-actions]

❌🪟 @autofix-ci[bot], there are 12 test regressions on Windows x86_64

• test\cli\inspect\inspect.test.ts
• test\js\bun\shell\bunshell-instance.test.ts
• test\js\bun\shell\bunshell.test.ts
• test\js\bun\shell\commands\rm.test.ts
• test\js\bun\shell\lazy.test.ts
• test\js\bun\shell\leak.test.ts
• test\js\bun\shell\lex.test.ts
• test\js\bun\util\stringWidth.test.ts
• test\js\node\events\events-cjs.test.js
• test\js\web\timers\performance.test.js
• test\regression\issue\08095.test.ts
• test\regression\issue\08258.test.ts

Full Test Output

[svemat01]

Any plans for this PR or is it ready to merge?