-
-
Notifications
You must be signed in to change notification settings - Fork 471
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for proxy #206
Comments
+1 |
This is due to using node-fetch which declined to add proxy support. It's quite a simple fix to use the octokit request API, which honours proxy settings and automatically supplies the access token: diff --git a/package.json b/package.json
index c0275f4..12e33a8 100644
--- a/package.json
+++ b/package.json
@@ -24,15 +24,13 @@
"@octokit/plugin-retry": "^4.0.3",
"@octokit/plugin-throttling": "^4.3.2",
"glob": "^8.0.3",
- "mime": "^3.0.0",
- "node-fetch": "^2.6.7"
+ "mime": "^3.0.0"
},
"devDependencies": {
"@types/glob": "^8.0.0",
"@types/jest": "^29.2.3",
"@types/mime": "^3.0.1",
"@types/node": "^18.11.9",
- "@types/node-fetch": "^2.5.12",
"@vercel/ncc": "^0.34.0",
"jest": "^29.3.1",
"jest-circus": "^29.3.1",
diff --git a/src/github.ts b/src/github.ts
index f4a0402..d91084c 100644
--- a/src/github.ts
+++ b/src/github.ts
@@ -1,4 +1,3 @@
-import fetch from "node-fetch";
import { GitHub } from "@actions/github/lib/utils";
import { Config, isTag, releaseBody } from "./util";
import { statSync, readFileSync } from "fs";
@@ -159,16 +158,16 @@ export const upload = async (
console.log(`⬆️ Uploading ${name}...`);
const endpoint = new URL(url);
endpoint.searchParams.append("name", name);
- const resp = await fetch(endpoint, {
+ const resp = await github.request({
+ method: "POST",
+ url: endpoint.toString(),
headers: {
"content-length": `${size}`,
"content-type": mime,
- authorization: `token ${config.github_token}`,
},
- method: "POST",
- body,
+ data: body,
});
- const json = await resp.json();
+ const json = resp.data;
if (resp.status !== 201) {
throw new Error(
`Failed to upload release asset ${name}. received status code ${ |
+1 |
Merged
softprops
added a commit
that referenced
this issue
Mar 8, 2024
* manually adopt and apply patch from #206 (comment) * rebuild --------- Co-authored-by: Doug Tangren <d.tangren@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
are the proxy environment variables supported when uploading the assets for the release?
I'm getting the following error and I'm an environment that uses an http/https proxy.
The release is created fine though, but the assets are missing.
The text was updated successfully, but these errors were encountered: