From c427ff9326c582722d73c9411da32baadf576b45 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=EC=8A=88=EB=A6=AC=ED=8A=AC?= <9804cjh@naver.com> Date: Mon, 13 Sep 2021 14:30:59 +0900 Subject: [PATCH] Remove useless error handling --- play-dl/YouTube/utils/request.ts | 41 ++++++++++++-------------------- 1 file changed, 15 insertions(+), 26 deletions(-) diff --git a/play-dl/YouTube/utils/request.ts b/play-dl/YouTube/utils/request.ts index 7010f35..976c487 100644 --- a/play-dl/YouTube/utils/request.ts +++ b/play-dl/YouTube/utils/request.ts @@ -1,5 +1,5 @@ import https, { RequestOptions } from 'https' -import {IncomingMessage } from 'http' +import { IncomingMessage } from 'http' import { URL } from 'url' interface RequestOpts extends RequestOptions{ @@ -21,7 +21,6 @@ async function https_getter(req_url : string, options : RequestOpts = {}): Promi let req = https.request(req_options, (response) => { resolve(response) }) - req.on('error', reject) if(options.method === "POST") req.write(options.body) req.end() }) @@ -29,36 +28,26 @@ async function https_getter(req_url : string, options : RequestOpts = {}): Promi export async function request(url : string, options? : RequestOpts): Promise{ return new Promise(async (resolve, reject) => { - try{ - let data = '' - let res = await https_getter(url, options) - if(Number(res.statusCode) >= 300 && Number(res.statusCode) < 400){ - res = await https_getter(res.headers.location as string , options) - } - else if(Number(res.statusCode) > 400){ - reject(`Got ${res.statusCode} from the request`) - } - res.setEncoding('utf-8') - res.on('data', (c) => data+=c) - res.on('end', () => resolve(data)) + let data = '' + let res = await https_getter(url, options) + if(Number(res.statusCode) >= 300 && Number(res.statusCode) < 400){ + res = await https_getter(res.headers.location as string , options) } - catch(err) { - reject(err) + else if(Number(res.statusCode) > 400){ + reject(`Got ${res.statusCode} from the request`) } + res.setEncoding('utf-8') + res.on('data', (c) => data+=c) + res.on('end', () => resolve(data)) }) } export async function request_stream(url : string, options? : RequestOpts): Promise{ return new Promise(async (resolve, reject) => { - try{ - let res = await https_getter(url, options) - if(Number(res.statusCode) >= 300 && Number(res.statusCode) < 400){ - res = await https_getter(res.headers.location as string, options) - } - resolve(res) - } - catch(err){ - reject(err) + let res = await https_getter(url, options) + if(Number(res.statusCode) >= 300 && Number(res.statusCode) < 400){ + res = await https_getter(res.headers.location as string, options) } + resolve(res) }) -} \ No newline at end of file +}