Skip to content

inspector: initial support for Network.loadNetworkResource #58077

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
nodejs-github-bot merged 5 commits into from
Jul 10, 2025
Merged

inspector: initial support for Network.loadNetworkResource #58077

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d99bb25
inspector: initial support for Network.loadNetworkResource
islandryu Apr 29, 2025
fbdcdcc
avoid making networkResourceManager a static class.
islandryu Jun 29, 2025
e3b871c
use url as stream id
islandryu Jun 30, 2025
34e9882
use std::optional
islandryu Jul 3, 2025
04d8eca
add flag to manpage
islandryu Jul 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
add flag to manpage
  • Loading branch information
@islandryu
islandryu committed Jul 9, 2025
commit 04d8ecac9633c41ec406e936d2c26c0c0a4daa62
3 changes: 3 additions & 0 deletions doc/node.1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -229,6 +229,9 @@ Enable experimental WebAssembly module support.
.It Fl -experimental-quic
Enable the experimental QUIC support.
.
.It Fl -experimental-inspector-network-resource
Enable experimental support for inspector network resources.
.
.It Fl -force-context-aware
Disable loading native addons that are not context-aware.
.
Expand Down
5 changes: 3 additions & 2 deletions test/parallel/test-inspector-network-resource.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,10 +14,11 @@ const resourceUrl = 'http://localhost:3000/app.js';
const resourcePath = path.join(__dirname, '../fixtures/inspector-network-resource/app.js.map');

const resourceText = fs.readFileSync(resourcePath, 'utf8');
const embedPath = resourcePath.replace(/\\/g, '\\\\').replace(/'/g, "\\'");
const script = `
const { NetworkResources } = require('node:inspector');
const fs = require('fs');
NetworkResources.put('${resourceUrl}', fs.readFileSync('${resourcePath.replace(/\\/g, '\\').replace(/'/g, "\\'")}', 'utf8'));
NetworkResources.put('${resourceUrl}', fs.readFileSync('${embedPath}', 'utf8'));
console.log('Network resource loaded:', '${resourceUrl}');
Comment thread
legendecas marked this conversation as resolved.
debugger;
`;
Expand Down Expand Up @@ -117,7 +118,7 @@ test('should load resource put from another thread', async () => {
const script = `
const { NetworkResources } = require('node:inspector');
const fs = require('fs');
Comment thread
legendecas marked this conversation as resolved.
NetworkResources.put('${resourceUrl}', fs.readFileSync('${resourcePath.replace(/\\/g, '\\').replace(/'/g, "\\'")}', 'utf8'));
NetworkResources.put('${resourceUrl}', fs.readFileSync('${embedPath}', 'utf8'));
const { Worker } = require('worker_threads');
const worker = new Worker(\`${workerScript}\`, {eval: true});
`;
Expand Down
Loading