Skip to content

Commit d2d34c5

Browse files
committed
reorder logic in getPreSignedURL to be happy
1 parent 140b3a1 commit d2d34c5

File tree

1 file changed

+12
-20
lines changed

1 file changed

+12
-20
lines changed

packages/bundler-plugin-core/src/utils/getPreSignedURL.ts

Lines changed: 12 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import { type ProviderServiceParams } from "../types.ts";
66
import { fetchWithRetry } from "./fetchWithRetry.ts";
77
import { green, red } from "./logging.ts";
88
import { preProcessBody } from "./preProcessBody.ts";
9-
import { NoUploadTokenError } from "../errors/NoUploadTokenError.ts";
109
import { findGitService } from "./findGitService.ts";
1110
import { UndefinedGitServiceError } from "../errors/UndefinedGitServiceError.ts";
1211
import { FailedOIDCFetchError } from "../errors/FailedOIDCFetchError.ts";
@@ -45,23 +44,7 @@ export const getPreSignedURL = async ({
4544
});
4645

4746
const requestBody: RequestBody = serviceParams;
48-
/**
49-
* We require the branch to be in the format `owner:branch` for tokenless v2 upload.
50-
* True Tokenless is allowed with any branch format.
51-
*/
52-
if (!uploadToken) {
53-
if (gitService) {
54-
requestBody.git_service = gitService;
55-
} else {
56-
const foundGitService = findGitService();
57-
if (!foundGitService || foundGitService === "") {
58-
red("Failed to find git service for tokenless upload");
59-
throw new UndefinedGitServiceError("No upload token provided");
60-
}
61-
62-
requestBody.git_service = foundGitService;
63-
}
64-
} else if (oidc?.useGitHubOIDC && Core) {
47+
if (oidc?.useGitHubOIDC && Core) {
6548
if (serviceParams?.service !== "github-actions") {
6649
red("OIDC is only supported for GitHub Actions");
6750
throw new BadOIDCServiceError(
@@ -88,8 +71,17 @@ export const getPreSignedURL = async ({
8871
} else if (uploadToken) {
8972
headers.set("Authorization", `token ${uploadToken}`);
9073
} else {
91-
red("No upload token provided");
92-
throw new NoUploadTokenError("No upload token provided");
74+
if (gitService) {
75+
requestBody.git_service = gitService;
76+
} else {
77+
const foundGitService = findGitService();
78+
if (!foundGitService || foundGitService === "") {
79+
red("Failed to find git service for tokenless upload");
80+
throw new UndefinedGitServiceError("No upload token provided");
81+
}
82+
83+
requestBody.git_service = foundGitService;
84+
}
9385
}
9486

9587
let response: Response;

0 commit comments

Comments
 (0)