build: fail publish when upload fatal errors (#48537)

We logged a fatal error but didn't exit with code 1 so the publish kept going. This was caught by a sanity check later down the release process but would have been quicker to fail out here.

Also adds some code to maybe workaround the underlying auth error
This commit is contained in:
Samuel Attard
2025-10-13 03:53:52 -07:00
committed by GitHub
parent dd25a6361b
commit 52929c93db

View File

@@ -48,7 +48,7 @@ const targetRepo = getRepo();
const uploadUrl = `https://uploads.github.com/repos/electron/${targetRepo}/releases/${releaseId}/assets{?name,label}`;
let retry = 0;
const octokit = new Octokit({
let octokit = new Octokit({
authStrategy: createGitHubTokenStrategy(targetRepo),
log: console
});
@@ -73,6 +73,12 @@ function uploadToGitHub () {
console.log(`Error uploading ${fileName} to GitHub, will retry. Error was:`, err);
retry++;
// Reset octokit in case it cached an auth error somehow
octokit = new Octokit({
authStrategy: createGitHubTokenStrategy(targetRepo),
log: console
});
octokit.repos.listReleaseAssets({
owner: ELECTRON_ORG,
repo: targetRepo,
@@ -98,6 +104,7 @@ function uploadToGitHub () {
}
}).catch((getReleaseErr) => {
console.log('Fatal: Unable to get current release assets via getRelease! Error was:', getReleaseErr);
process.exitCode = 1;
});
} else {
console.log(`Error retrying uploading ${fileName} to GitHub:`, err);