Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Antoine du Hamel <[email protected]>
  • Loading branch information
GeoffreyBooth and aduh95 committed Sep 14, 2023
1 parent da76e59 commit e564349
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions lib/internal/modules/esm/fetch_module.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ const cacheForGET = new SafeMap();
/** @type {import('https').Agent} The Cached HTTP Agent for **secure** HTTP requests. */
let HTTPSAgent;
/**
* Make a **secure** HTTP GET request (handling agent setup if needed, caching the agent to avoid
* Make a HTTPs GET request (handling agent setup if needed, caching the agent to avoid
* redudant instantiations).
* @param {Parameters<import('https').get>[0]} input - The URI to fetch.
* @param {Parameters<import('https').get>[1]} options - See https.get() options.
Expand All @@ -66,10 +66,10 @@ function HTTPSGet(input, options) {
/** @type {import('https').Agent} The Cached HTTP Agent for **insecure** HTTP requests. */
let HTTPAgent;
/**
* Make a **insecure** HTTP GET request (handling agent setup if needed, caching the agent to avoid
* Make a HTTP GET request (handling agent setup if needed, caching the agent to avoid
* redudant instantiations).
* @param {Parameters<import('https').get>[0]} input - The URI to fetch.
* @param {Parameters<import('https').get>[1]} options - See http.get() options.
* @param {Parameters<import('http').get>[0]} input - The URI to fetch.
* @param {Parameters<import('http').get>[1]} options - See http.get() options.
*/
function HTTPGet(input, options) {
const http = require('http'); // [1]
Expand Down

0 comments on commit e564349

Please sign in to comment.