Skip to content

Commit 72f6a9d

Browse files
authored
Merge pull request #4030 from github/mbg/fix/no-proxy
Use default `request` options instead of `undefined`
2 parents bfb6be4 + 3b5ee58 commit 72f6a9d

2 files changed

Lines changed: 66 additions & 64 deletions

File tree

lib/entry-points.js

Lines changed: 63 additions & 63 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/api-client.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,9 @@ function createApiClientWithDetails(
137137
(allowExternal && apiDetails.externalRepoAuth) || apiDetails.auth;
138138
const retryingOctokit = githubUtils.GitHub.plugin(retry.retry);
139139
const requestOptions =
140-
proxy === undefined ? undefined : makeProxyRequestOptions(proxy);
140+
proxy === undefined
141+
? githubUtils.defaults.request
142+
: makeProxyRequestOptions(proxy);
141143
return new retryingOctokit(
142144
githubUtils.getOctokitOptions(auth, {
143145
baseUrl: apiDetails.apiURL,

0 commit comments

Comments
 (0)