Deno/deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-11-21 20:38:55 +00:00
Code Issues Actions 4 Packages Projects Releases Wiki Activity

Revert "fix(ext/node): use primordials in ext/node/polyfills/https.ts (#26323)" (#26613)

Browse Source 
…s` (#26323)"

This reverts commit afb33b3c25.

Reverting because it caused a regression -
https://github.com/denoland/deno/issues/26612.

Closes https://github.com/denoland/deno/issues/26612.
This commit is contained in:
Bartek Iwańczuk 2024-10-29 00:41:02 +00:00 committed by GitHub
parent 484f8ca9c3
commit 46e5ed1a64
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 12 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
ext/node/polyfills/https.ts
View File

@ -1,6 +1,9 @@
// Copyright 2018-2024 the Deno authors. All rights reserved. MIT license.
// Copyright Joyent and Node contributors. All rights reserved. MIT license.
// TODO(petamoriken): enable prefer-primordials for node polyfills
// deno-lint-ignore-file prefer-primordials
import { notImplemented } from "ext:deno_node/_utils.ts";
import { urlToHttpOptions } from "ext:deno_node/internal/url.ts";
import {
@ -14,14 +17,6 @@ import { type ServerHandler, ServerImpl as HttpServer } from "node:http";
import { validateObject } from "ext:deno_node/internal/validators.mjs";
import { kEmptyObject } from "ext:deno_node/internal/util.mjs";
import { Buffer } from "node:buffer";
import { primordials } from "ext:core/mod.js";
const {
ArrayPrototypeShift,
ArrayPrototypeUnshift,
ArrayIsArray,
ObjectPrototypeIsPrototypeOf,
ObjectAssign,
} = primordials;
export class Server extends HttpServer {
constructor(opts, requestListener?: ServerHandler) {
@ -34,11 +29,11 @@ export class Server extends HttpServer {
validateObject(opts, "options");
}
if (opts.cert && ArrayIsArray(opts.cert)) {
if (opts.cert && Array.isArray(opts.cert)) {
notImplemented("https.Server.opts.cert array type");
}
if (opts.key && ArrayIsArray(opts.key)) {
if (opts.key && Array.isArray(opts.key)) {
notImplemented("https.Server.opts.key array type");
}
@ -47,12 +42,10 @@ export class Server extends HttpServer {
_additionalServeOptions() {
return {
cert: ObjectPrototypeIsPrototypeOf(Buffer, this._opts.cert)
// deno-lint-ignore prefer-primordials
cert: this._opts.cert instanceof Buffer
? this._opts.cert.toString()
: this._opts.cert,
key: ObjectPrototypeIsPrototypeOf(Buffer, this._opts.key)
// deno-lint-ignore prefer-primordials
key: this._opts.key instanceof Buffer
? this._opts.key.toString()
: this._opts.key,
};
@ -166,18 +159,18 @@ export function request(...args: any[]) {
let options = {};
if (typeof args[0] === "string") {
const urlStr = ArrayPrototypeShift(args);
const urlStr = args.shift();
options = urlToHttpOptions(new URL(urlStr));
} else if (ObjectPrototypeIsPrototypeOf(URL, args[0])) {
options = urlToHttpOptions(ArrayPrototypeShift(args));
} else if (args[0] instanceof URL) {
options = urlToHttpOptions(args.shift());
}
if (args[0] && typeof args[0] !== "function") {
ObjectAssign(options, ArrayPrototypeShift(args));
Object.assign(options, args.shift());
}
options._defaultAgent = globalAgent;
ArrayPrototypeUnshift(args, options);
args.unshift(options);
return new HttpsClientRequest(args[0], args[1]);
}