@octokit/request
Advanced tools
Comparing version 6.2.2 to 6.2.3
@@ -13,3 +13,3 @@ 'use strict'; | ||
const VERSION = "6.2.2"; | ||
const VERSION = "6.2.3"; | ||
@@ -22,13 +22,9 @@ function getBufferResponse(response) { | ||
const log = requestOptions.request && requestOptions.request.log ? requestOptions.request.log : console; | ||
if (isPlainObject.isPlainObject(requestOptions.body) || Array.isArray(requestOptions.body)) { | ||
requestOptions.body = JSON.stringify(requestOptions.body); | ||
} | ||
let headers = {}; | ||
let status; | ||
let url; | ||
const fetch = requestOptions.request && requestOptions.request.fetch || globalThis.fetch || | ||
/* istanbul ignore next */ | ||
nodeFetch; | ||
const fetch = requestOptions.request && requestOptions.request.fetch || globalThis.fetch || /* istanbul ignore next */nodeFetch; | ||
return fetch(requestOptions.url, Object.assign({ | ||
@@ -39,3 +35,4 @@ method: requestOptions.method, | ||
redirect: requestOptions.redirect | ||
}, // `requestOptions.request.agent` type is incompatible | ||
}, | ||
// `requestOptions.request.agent` type is incompatible | ||
// see https://github.com/octokit/types.ts/pull/264 | ||
@@ -45,7 +42,5 @@ requestOptions.request)).then(async response => { | ||
status = response.status; | ||
for (const keyAndValue of response.headers) { | ||
headers[keyAndValue[0]] = keyAndValue[1]; | ||
} | ||
if ("deprecation" in headers) { | ||
@@ -56,8 +51,6 @@ const matches = headers.link && headers.link.match(/<([^>]+)>; rel="deprecation"/); | ||
} | ||
if (status === 204 || status === 205) { | ||
return; | ||
} // GitHub API returns 200 for HEAD requests | ||
} | ||
// GitHub API returns 200 for HEAD requests | ||
if (requestOptions.method === "HEAD") { | ||
@@ -67,3 +60,2 @@ if (status < 400) { | ||
} | ||
throw new requestError.RequestError(response.statusText, status, { | ||
@@ -79,3 +71,2 @@ response: { | ||
} | ||
if (status === 304) { | ||
@@ -92,3 +83,2 @@ throw new requestError.RequestError("Not modified", status, { | ||
} | ||
if (status >= 400) { | ||
@@ -107,3 +97,2 @@ const data = await getResponseData(response); | ||
} | ||
return getResponseData(response); | ||
@@ -124,20 +113,15 @@ }).then(data => { | ||
} | ||
async function getResponseData(response) { | ||
const contentType = response.headers.get("content-type"); | ||
if (/application\/json/.test(contentType)) { | ||
return response.json(); | ||
} | ||
if (!contentType || /^text\/|charset=utf-8$/.test(contentType)) { | ||
return response.text(); | ||
} | ||
return getBufferResponse(response); | ||
} | ||
function toErrorMessage(data) { | ||
if (typeof data === "string") return data; // istanbul ignore else - just in case | ||
if (typeof data === "string") return data; | ||
// istanbul ignore else - just in case | ||
if ("message" in data) { | ||
@@ -147,7 +131,5 @@ if (Array.isArray(data.errors)) { | ||
} | ||
return data.message; | ||
} // istanbul ignore next - just in case | ||
} | ||
// istanbul ignore next - just in case | ||
return `Unknown error: ${JSON.stringify(data)}`; | ||
@@ -158,14 +140,10 @@ } | ||
const endpoint = oldEndpoint.defaults(newDefaults); | ||
const newApi = function (route, parameters) { | ||
const endpointOptions = endpoint.merge(route, parameters); | ||
if (!endpointOptions.request || !endpointOptions.request.hook) { | ||
return fetchWrapper(endpoint.parse(endpointOptions)); | ||
} | ||
const request = (route, parameters) => { | ||
return fetchWrapper(endpoint.parse(endpoint.merge(route, parameters))); | ||
}; | ||
Object.assign(request, { | ||
@@ -177,3 +155,2 @@ endpoint, | ||
}; | ||
return Object.assign(newApi, { | ||
@@ -180,0 +157,0 @@ endpoint, |
@@ -1,1 +0,1 @@ | ||
export const VERSION = "6.2.2"; | ||
export const VERSION = "6.2.3"; |
@@ -1,1 +0,1 @@ | ||
export declare const VERSION = "6.2.2"; | ||
export declare const VERSION = "6.2.3"; |
@@ -7,3 +7,3 @@ import { endpoint } from '@octokit/endpoint'; | ||
const VERSION = "6.2.2"; | ||
const VERSION = "6.2.3"; | ||
@@ -10,0 +10,0 @@ function getBufferResponse(response) { |
{ | ||
"name": "@octokit/request", | ||
"description": "Send parameterized requests to GitHub's APIs with sensible defaults in browsers and Node", | ||
"version": "6.2.2", | ||
"version": "6.2.3", | ||
"license": "MIT", | ||
@@ -26,3 +26,3 @@ "files": [ | ||
"@octokit/request-error": "^3.0.0", | ||
"@octokit/types": "^8.0.0", | ||
"@octokit/types": "^9.0.0", | ||
"is-plain-object": "^5.0.0", | ||
@@ -41,3 +41,3 @@ "node-fetch": "^2.6.7", | ||
"@types/lolex": "^5.1.0", | ||
"@types/node": "^16.0.0", | ||
"@types/node": "^18.0.0", | ||
"@types/node-fetch": "^2.3.3", | ||
@@ -48,3 +48,3 @@ "@types/once": "^1.4.0", | ||
"lolex": "^6.0.0", | ||
"prettier": "2.7.1", | ||
"prettier": "2.8.3", | ||
"semantic-release-plugin-update-version-in-files": "^1.0.0", | ||
@@ -51,0 +51,0 @@ "string-to-arraybuffer": "^1.0.2", |
@@ -6,3 +6,3 @@ # request.js | ||
[![@latest](https://img.shields.io/npm/v/@octokit/request.svg)](https://www.npmjs.com/package/@octokit/request) | ||
[![Build Status](https://github.com/octokit/request.js/workflows/Test/badge.svg)](https://github.com/octokit/request.js/actions?query=workflow%3ATest+branch%3Amaster) | ||
[![Build Status](https://github.com/octokit/request.js/workflows/Test/badge.svg)](https://github.com/octokit/request.js/actions?query=workflow%3ATest+branch%3Amain) | ||
@@ -75,6 +75,6 @@ `@octokit/request` is a request library for browsers & node that makes it easier | ||
Load <code>@octokit/request</code> directly from <a href="https://cdn.skypack.dev">cdn.skypack.dev</a> | ||
```html | ||
<script type="module"> | ||
import { request } from "https://cdn.skypack.dev/@octokit/request"; | ||
import { request } from "https://cdn.skypack.dev/@octokit/request"; | ||
</script> | ||
@@ -81,0 +81,0 @@ ``` |
Sorry, the diff of this file is not supported yet
Sorry, the diff of this file is not supported yet
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
478
58010
- Removed@octokit/openapi-types@14.0.0(transitive)
- Removed@octokit/types@8.2.1(transitive)
Updated@octokit/types@^9.0.0