mirror of
https://github.com/game-ci/unity-builder.git
synced 2026-06-01 14:26:17 -07:00
Compare commits
4 Commits
feature/or
...
fix/secure
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
f80e4f66d4 | ||
|
|
b2327008ed | ||
|
|
b3bd405399 | ||
|
|
8a41533779 |
9
.github/workflows/orchestrator-integrity.yml
vendored
9
.github/workflows/orchestrator-integrity.yml
vendored
@@ -196,15 +196,6 @@ jobs:
|
|||||||
fi
|
fi
|
||||||
- run: yarn install --frozen-lockfile
|
- run: yarn install --frozen-lockfile
|
||||||
# ==========================================
|
# ==========================================
|
||||||
# FAST UNIT TESTS (no infra required, fast-fail gate)
|
|
||||||
# ==========================================
|
|
||||||
- name: Run orchestrator unit tests (fast, no infra)
|
|
||||||
timeout-minutes: 2
|
|
||||||
run: >-
|
|
||||||
yarn run test
|
|
||||||
--testPathPattern="orchestrator-guid|orchestrator-folders|task-parameter-serializer|follow-log-stream-service|runner-availability-service|provider-url-parser|provider-loader|provider-git-manager|orchestrator-image|orchestrator-hooks|orchestrator-github-checks"
|
|
||||||
--verbose --detectOpenHandles --forceExit --runInBand
|
|
||||||
# ==========================================
|
|
||||||
# K8S TESTS SECTION
|
# K8S TESTS SECTION
|
||||||
# ==========================================
|
# ==========================================
|
||||||
- name: Clean up disk space before K8s tests
|
- name: Clean up disk space before K8s tests
|
||||||
|
|||||||
@@ -105,6 +105,12 @@ inputs:
|
|||||||
required: false
|
required: false
|
||||||
default: ''
|
default: ''
|
||||||
description: '[Orchestrator] Github private token to pull from github'
|
description: '[Orchestrator] Github private token to pull from github'
|
||||||
|
gitAuthMode:
|
||||||
|
required: false
|
||||||
|
default: 'header'
|
||||||
|
description:
|
||||||
|
'[Orchestrator] How git authentication is configured. "header" (default) uses http.extraHeader so the token
|
||||||
|
never appears in clone URLs or git config. "url" embeds the token in clone URLs (legacy behavior).'
|
||||||
githubOwner:
|
githubOwner:
|
||||||
required: false
|
required: false
|
||||||
default: ''
|
default: ''
|
||||||
|
|||||||
102
dist/index.js
generated
vendored
102
dist/index.js
generated
vendored
@@ -327,6 +327,7 @@ class BuildParameters {
|
|||||||
containerRegistryRepository: input_1.default.containerRegistryRepository,
|
containerRegistryRepository: input_1.default.containerRegistryRepository,
|
||||||
containerRegistryImageVersion: input_1.default.containerRegistryImageVersion,
|
containerRegistryImageVersion: input_1.default.containerRegistryImageVersion,
|
||||||
providerStrategy: orchestrator_options_1.default.providerStrategy,
|
providerStrategy: orchestrator_options_1.default.providerStrategy,
|
||||||
|
gitAuthMode: orchestrator_options_1.default.gitAuthMode,
|
||||||
buildPlatform: orchestrator_options_1.default.buildPlatform,
|
buildPlatform: orchestrator_options_1.default.buildPlatform,
|
||||||
kubeConfig: orchestrator_options_1.default.kubeConfig,
|
kubeConfig: orchestrator_options_1.default.kubeConfig,
|
||||||
containerMemory: orchestrator_options_1.default.containerMemory,
|
containerMemory: orchestrator_options_1.default.containerMemory,
|
||||||
@@ -1944,6 +1945,29 @@ exports["default"] = OrchestratorEnvironmentVariable;
|
|||||||
|
|
||||||
"use strict";
|
"use strict";
|
||||||
|
|
||||||
|
var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
|
||||||
|
if (k2 === undefined) k2 = k;
|
||||||
|
var desc = Object.getOwnPropertyDescriptor(m, k);
|
||||||
|
if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
|
||||||
|
desc = { enumerable: true, get: function() { return m[k]; } };
|
||||||
|
}
|
||||||
|
Object.defineProperty(o, k2, desc);
|
||||||
|
}) : (function(o, m, k, k2) {
|
||||||
|
if (k2 === undefined) k2 = k;
|
||||||
|
o[k2] = m[k];
|
||||||
|
}));
|
||||||
|
var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
|
||||||
|
Object.defineProperty(o, "default", { enumerable: true, value: v });
|
||||||
|
}) : function(o, v) {
|
||||||
|
o["default"] = v;
|
||||||
|
});
|
||||||
|
var __importStar = (this && this.__importStar) || function (mod) {
|
||||||
|
if (mod && mod.__esModule) return mod;
|
||||||
|
var result = {};
|
||||||
|
if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
|
||||||
|
__setModuleDefault(result, mod);
|
||||||
|
return result;
|
||||||
|
};
|
||||||
var __importDefault = (this && this.__importDefault) || function (mod) {
|
var __importDefault = (this && this.__importDefault) || function (mod) {
|
||||||
return (mod && mod.__esModule) ? mod : { "default": mod };
|
return (mod && mod.__esModule) ? mod : { "default": mod };
|
||||||
};
|
};
|
||||||
@@ -1998,12 +2022,57 @@ class OrchestratorFolders {
|
|||||||
static get libraryCacheFolderFull() {
|
static get libraryCacheFolderFull() {
|
||||||
return node_path_1.default.join(OrchestratorFolders.cacheFolderForCacheKeyFull, `Library`);
|
return node_path_1.default.join(OrchestratorFolders.cacheFolderForCacheKeyFull, `Library`);
|
||||||
}
|
}
|
||||||
|
/**
|
||||||
|
* Whether to use http.extraHeader for git authentication (secure, default)
|
||||||
|
* instead of embedding the token in clone URLs (legacy).
|
||||||
|
*/
|
||||||
|
static get useHeaderAuth() {
|
||||||
|
return orchestrator_1.default.buildParameters.gitAuthMode !== 'url';
|
||||||
|
}
|
||||||
static get unityBuilderRepoUrl() {
|
static get unityBuilderRepoUrl() {
|
||||||
|
if (OrchestratorFolders.useHeaderAuth) {
|
||||||
|
return `https://github.com/${orchestrator_1.default.buildParameters.orchestratorRepoName}.git`;
|
||||||
|
}
|
||||||
return `https://${orchestrator_1.default.buildParameters.gitPrivateToken}@github.com/${orchestrator_1.default.buildParameters.orchestratorRepoName}.git`;
|
return `https://${orchestrator_1.default.buildParameters.gitPrivateToken}@github.com/${orchestrator_1.default.buildParameters.orchestratorRepoName}.git`;
|
||||||
}
|
}
|
||||||
static get targetBuildRepoUrl() {
|
static get targetBuildRepoUrl() {
|
||||||
|
if (OrchestratorFolders.useHeaderAuth) {
|
||||||
|
return `https://github.com/${orchestrator_1.default.buildParameters.githubRepo}.git`;
|
||||||
|
}
|
||||||
return `https://${orchestrator_1.default.buildParameters.gitPrivateToken}@github.com/${orchestrator_1.default.buildParameters.githubRepo}.git`;
|
return `https://${orchestrator_1.default.buildParameters.gitPrivateToken}@github.com/${orchestrator_1.default.buildParameters.githubRepo}.git`;
|
||||||
}
|
}
|
||||||
|
/**
|
||||||
|
* Shell commands to configure git authentication via http.extraHeader.
|
||||||
|
* Uses GIT_PRIVATE_TOKEN env var so the token never appears in clone URLs or git config output.
|
||||||
|
* This is the same mechanism used by actions/checkout.
|
||||||
|
*
|
||||||
|
* Only emits commands when gitAuthMode is 'header' (default). In 'url' mode,
|
||||||
|
* returns a no-op comment since the token is already in the URL.
|
||||||
|
*/
|
||||||
|
static get gitAuthConfigScript() {
|
||||||
|
if (!OrchestratorFolders.useHeaderAuth) {
|
||||||
|
return `# git auth: using token-in-URL mode (legacy)`;
|
||||||
|
}
|
||||||
|
return `# git auth: configuring http.extraHeader (secure mode)
|
||||||
|
if [ -n "$GIT_PRIVATE_TOKEN" ]; then
|
||||||
|
git config --global http.https://github.com/.extraHeader "Authorization: Basic $(printf '%s' "x-access-token:$GIT_PRIVATE_TOKEN" | base64 -w 0)"
|
||||||
|
fi`;
|
||||||
|
}
|
||||||
|
/**
|
||||||
|
* Configure git authentication via http.extraHeader in the current Node process.
|
||||||
|
* For use in the remote-client where shell scripts aren't used.
|
||||||
|
* Only configures when gitAuthMode is 'header' (default).
|
||||||
|
*/
|
||||||
|
static async configureGitAuth() {
|
||||||
|
if (!OrchestratorFolders.useHeaderAuth)
|
||||||
|
return;
|
||||||
|
const token = orchestrator_1.default.buildParameters.gitPrivateToken || process.env.GIT_PRIVATE_TOKEN || '';
|
||||||
|
if (!token)
|
||||||
|
return;
|
||||||
|
const encoded = Buffer.from(`x-access-token:${token}`).toString('base64');
|
||||||
|
const { OrchestratorSystem } = await Promise.resolve().then(() => __importStar(__nccwpck_require__(9744)));
|
||||||
|
await OrchestratorSystem.Run(`git config --global http.https://github.com/.extraHeader "Authorization: Basic ${encoded}"`);
|
||||||
|
}
|
||||||
static get buildVolumeFolder() {
|
static get buildVolumeFolder() {
|
||||||
return 'data';
|
return 'data';
|
||||||
}
|
}
|
||||||
@@ -2204,6 +2273,9 @@ class OrchestratorOptions {
|
|||||||
}
|
}
|
||||||
return provider || 'local';
|
return provider || 'local';
|
||||||
}
|
}
|
||||||
|
static get gitAuthMode() {
|
||||||
|
return OrchestratorOptions.getInput('gitAuthMode') || 'header';
|
||||||
|
}
|
||||||
static get containerCpu() {
|
static get containerCpu() {
|
||||||
return OrchestratorOptions.getInput('containerCpu') || `1024`;
|
return OrchestratorOptions.getInput('containerCpu') || `1024`;
|
||||||
}
|
}
|
||||||
@@ -7985,6 +8057,7 @@ class RemoteClient {
|
|||||||
}
|
}
|
||||||
remote_client_logger_1.RemoteClientLogger.log(`Initializing source repository for cloning with caching of LFS files`);
|
remote_client_logger_1.RemoteClientLogger.log(`Initializing source repository for cloning with caching of LFS files`);
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global advice.detachedHead false`);
|
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global advice.detachedHead false`);
|
||||||
|
await orchestrator_folders_1.OrchestratorFolders.configureGitAuth();
|
||||||
remote_client_logger_1.RemoteClientLogger.log(`Cloning the repository being built:`);
|
remote_client_logger_1.RemoteClientLogger.log(`Cloning the repository being built:`);
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global filter.lfs.smudge "git-lfs smudge --skip -- %f"`);
|
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global filter.lfs.smudge "git-lfs smudge --skip -- %f"`);
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global filter.lfs.process "git-lfs filter-process --skip"`);
|
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global filter.lfs.process "git-lfs filter-process --skip"`);
|
||||||
@@ -8089,10 +8162,7 @@ class RemoteClient {
|
|||||||
const gitPrivateToken = process.env.GIT_PRIVATE_TOKEN;
|
const gitPrivateToken = process.env.GIT_PRIVATE_TOKEN;
|
||||||
if (gitPrivateToken) {
|
if (gitPrivateToken) {
|
||||||
remote_client_logger_1.RemoteClientLogger.log(`Attempting to pull LFS files with GIT_PRIVATE_TOKEN...`);
|
remote_client_logger_1.RemoteClientLogger.log(`Attempting to pull LFS files with GIT_PRIVATE_TOKEN...`);
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global --unset-all url."https://github.com/".insteadOf || true`);
|
await RemoteClient.configureTokenAuth(gitPrivateToken);
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global --unset-all url."ssh://git@github.com/".insteadOf || true`);
|
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global --unset-all url."git@github.com".insteadOf || true`);
|
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global url."https://${gitPrivateToken}@github.com/".insteadOf "https://github.com/"`);
|
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git lfs pull`, true);
|
await orchestrator_system_1.OrchestratorSystem.Run(`git lfs pull`, true);
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git lfs checkout || true`, true);
|
await orchestrator_system_1.OrchestratorSystem.Run(`git lfs checkout || true`, true);
|
||||||
remote_client_logger_1.RemoteClientLogger.log(`Successfully pulled LFS files with GIT_PRIVATE_TOKEN`);
|
remote_client_logger_1.RemoteClientLogger.log(`Successfully pulled LFS files with GIT_PRIVATE_TOKEN`);
|
||||||
@@ -8107,10 +8177,7 @@ class RemoteClient {
|
|||||||
const githubToken = process.env.GITHUB_TOKEN;
|
const githubToken = process.env.GITHUB_TOKEN;
|
||||||
if (githubToken) {
|
if (githubToken) {
|
||||||
remote_client_logger_1.RemoteClientLogger.log(`Attempting to pull LFS files with GITHUB_TOKEN fallback...`);
|
remote_client_logger_1.RemoteClientLogger.log(`Attempting to pull LFS files with GITHUB_TOKEN fallback...`);
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global --unset-all url."https://github.com/".insteadOf || true`);
|
await RemoteClient.configureTokenAuth(githubToken);
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global --unset-all url."ssh://git@github.com/".insteadOf || true`);
|
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global --unset-all url."git@github.com".insteadOf || true`);
|
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global url."https://${githubToken}@github.com/".insteadOf "https://github.com/"`);
|
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git lfs pull`, true);
|
await orchestrator_system_1.OrchestratorSystem.Run(`git lfs pull`, true);
|
||||||
await orchestrator_system_1.OrchestratorSystem.Run(`git lfs checkout || true`, true);
|
await orchestrator_system_1.OrchestratorSystem.Run(`git lfs checkout || true`, true);
|
||||||
remote_client_logger_1.RemoteClientLogger.log(`Successfully pulled LFS files with GITHUB_TOKEN`);
|
remote_client_logger_1.RemoteClientLogger.log(`Successfully pulled LFS files with GITHUB_TOKEN`);
|
||||||
@@ -8167,6 +8234,23 @@ class RemoteClient {
|
|||||||
}
|
}
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
/**
|
||||||
|
* Configure git authentication for a token. In header mode (default), uses
|
||||||
|
* http.extraHeader so the token never appears in URLs or git config output.
|
||||||
|
* In url mode (legacy), uses url.insteadOf to embed the token in URLs.
|
||||||
|
*/
|
||||||
|
static async configureTokenAuth(token) {
|
||||||
|
if (orchestrator_folders_1.OrchestratorFolders.useHeaderAuth) {
|
||||||
|
const encoded = Buffer.from(`x-access-token:${token}`).toString('base64');
|
||||||
|
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global http.https://github.com/.extraHeader "Authorization: Basic ${encoded}"`);
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global --unset-all url."https://github.com/".insteadOf || true`);
|
||||||
|
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global --unset-all url."ssh://git@github.com/".insteadOf || true`);
|
||||||
|
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global --unset-all url."git@github.com".insteadOf || true`);
|
||||||
|
await orchestrator_system_1.OrchestratorSystem.Run(`git config --global url."https://${token}@github.com/".insteadOf "https://github.com/"`);
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
__decorate([
|
__decorate([
|
||||||
(0, cli_functions_repository_1.CliFunction)(`remote-cli-pre-build`, `sets up a repository, usually before a game-ci build`)
|
(0, cli_functions_repository_1.CliFunction)(`remote-cli-pre-build`, `sets up a repository, usually before a game-ci build`)
|
||||||
@@ -9725,6 +9809,7 @@ printenv
|
|||||||
git config --global advice.detachedHead false
|
git config --global advice.detachedHead false
|
||||||
git config --global filter.lfs.smudge "git-lfs smudge --skip -- %f"
|
git config --global filter.lfs.smudge "git-lfs smudge --skip -- %f"
|
||||||
git config --global filter.lfs.process "git-lfs filter-process --skip"
|
git config --global filter.lfs.process "git-lfs filter-process --skip"
|
||||||
|
${orchestrator_folders_1.OrchestratorFolders.gitAuthConfigScript}
|
||||||
BRANCH="${orchestrator_1.default.buildParameters.orchestratorBranch}"
|
BRANCH="${orchestrator_1.default.buildParameters.orchestratorBranch}"
|
||||||
REPO="${orchestrator_folders_1.OrchestratorFolders.unityBuilderRepoUrl}"
|
REPO="${orchestrator_folders_1.OrchestratorFolders.unityBuilderRepoUrl}"
|
||||||
if [ -n "$(git ls-remote --heads "$REPO" "$BRANCH" 2>/dev/null)" ]; then
|
if [ -n "$(git ls-remote --heads "$REPO" "$BRANCH" 2>/dev/null)" ]; then
|
||||||
@@ -9841,6 +9926,7 @@ class BuildAutomationWorkflow {
|
|||||||
static setupCommands(builderPath, isContainerized) {
|
static setupCommands(builderPath, isContainerized) {
|
||||||
// prettier-ignore
|
// prettier-ignore
|
||||||
const commands = `mkdir -p ${orchestrator_folders_1.OrchestratorFolders.ToLinuxFolder(orchestrator_folders_1.OrchestratorFolders.builderPathAbsolute)}
|
const commands = `mkdir -p ${orchestrator_folders_1.OrchestratorFolders.ToLinuxFolder(orchestrator_folders_1.OrchestratorFolders.builderPathAbsolute)}
|
||||||
|
${orchestrator_folders_1.OrchestratorFolders.gitAuthConfigScript}
|
||||||
BRANCH="${orchestrator_1.default.buildParameters.orchestratorBranch}"
|
BRANCH="${orchestrator_1.default.buildParameters.orchestratorBranch}"
|
||||||
REPO="${orchestrator_folders_1.OrchestratorFolders.unityBuilderRepoUrl}"
|
REPO="${orchestrator_folders_1.OrchestratorFolders.unityBuilderRepoUrl}"
|
||||||
DEST="${orchestrator_folders_1.OrchestratorFolders.ToLinuxFolder(orchestrator_folders_1.OrchestratorFolders.builderPathAbsolute)}"
|
DEST="${orchestrator_folders_1.OrchestratorFolders.ToLinuxFolder(orchestrator_folders_1.OrchestratorFolders.builderPathAbsolute)}"
|
||||||
|
|||||||
2
dist/index.js.map
generated
vendored
2
dist/index.js.map
generated
vendored
File diff suppressed because one or more lines are too long
@@ -54,6 +54,7 @@ class BuildParameters {
|
|||||||
public sshAgent!: string;
|
public sshAgent!: string;
|
||||||
public sshPublicKeysDirectoryPath!: string;
|
public sshPublicKeysDirectoryPath!: string;
|
||||||
public providerStrategy!: string;
|
public providerStrategy!: string;
|
||||||
|
public gitAuthMode!: string;
|
||||||
public gitPrivateToken!: string;
|
public gitPrivateToken!: string;
|
||||||
public awsStackName!: string;
|
public awsStackName!: string;
|
||||||
public awsEndpoint?: string;
|
public awsEndpoint?: string;
|
||||||
@@ -194,6 +195,7 @@ class BuildParameters {
|
|||||||
containerRegistryRepository: Input.containerRegistryRepository,
|
containerRegistryRepository: Input.containerRegistryRepository,
|
||||||
containerRegistryImageVersion: Input.containerRegistryImageVersion,
|
containerRegistryImageVersion: Input.containerRegistryImageVersion,
|
||||||
providerStrategy: OrchestratorOptions.providerStrategy,
|
providerStrategy: OrchestratorOptions.providerStrategy,
|
||||||
|
gitAuthMode: OrchestratorOptions.gitAuthMode,
|
||||||
buildPlatform: OrchestratorOptions.buildPlatform,
|
buildPlatform: OrchestratorOptions.buildPlatform,
|
||||||
kubeConfig: OrchestratorOptions.kubeConfig,
|
kubeConfig: OrchestratorOptions.kubeConfig,
|
||||||
containerMemory: OrchestratorOptions.containerMemory,
|
containerMemory: OrchestratorOptions.containerMemory,
|
||||||
|
|||||||
140
src/model/orchestrator/options/orchestrator-folders-auth.test.ts
Normal file
140
src/model/orchestrator/options/orchestrator-folders-auth.test.ts
Normal file
@@ -0,0 +1,140 @@
|
|||||||
|
import { OrchestratorFolders } from './orchestrator-folders';
|
||||||
|
|
||||||
|
jest.mock('../orchestrator', () => ({
|
||||||
|
__esModule: true,
|
||||||
|
default: {
|
||||||
|
buildParameters: {
|
||||||
|
orchestratorRepoName: 'game-ci/unity-builder',
|
||||||
|
githubRepo: 'myorg/myrepo',
|
||||||
|
gitPrivateToken: 'ghp_test123',
|
||||||
|
gitAuthMode: 'header',
|
||||||
|
buildGuid: 'test-guid',
|
||||||
|
projectPath: '',
|
||||||
|
buildPath: 'Builds',
|
||||||
|
cacheKey: 'test-cache',
|
||||||
|
},
|
||||||
|
lockedWorkspace: '',
|
||||||
|
},
|
||||||
|
}));
|
||||||
|
|
||||||
|
jest.mock('./orchestrator-options', () => ({
|
||||||
|
__esModule: true,
|
||||||
|
default: {
|
||||||
|
useSharedBuilder: false,
|
||||||
|
},
|
||||||
|
}));
|
||||||
|
|
||||||
|
jest.mock('../services/core/orchestrator-system', () => ({
|
||||||
|
OrchestratorSystem: {
|
||||||
|
Run: jest.fn().mockResolvedValue(''),
|
||||||
|
},
|
||||||
|
}));
|
||||||
|
|
||||||
|
const mockOrchestrator = require('../orchestrator').default;
|
||||||
|
|
||||||
|
describe('OrchestratorFolders git auth', () => {
|
||||||
|
beforeEach(() => {
|
||||||
|
jest.clearAllMocks();
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('useHeaderAuth', () => {
|
||||||
|
it('should return true when gitAuthMode is header', () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'header';
|
||||||
|
expect(OrchestratorFolders.useHeaderAuth).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return true when gitAuthMode is undefined (default)', () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = undefined;
|
||||||
|
expect(OrchestratorFolders.useHeaderAuth).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return false when gitAuthMode is url', () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'url';
|
||||||
|
expect(OrchestratorFolders.useHeaderAuth).toBe(false);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('unityBuilderRepoUrl', () => {
|
||||||
|
it('should not include token in URL when using header auth', () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'header';
|
||||||
|
const url = OrchestratorFolders.unityBuilderRepoUrl;
|
||||||
|
expect(url).toBe('https://github.com/game-ci/unity-builder.git');
|
||||||
|
expect(url).not.toContain('ghp_test123');
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should include token in URL when using url auth (legacy)', () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'url';
|
||||||
|
const url = OrchestratorFolders.unityBuilderRepoUrl;
|
||||||
|
expect(url).toBe('https://ghp_test123@github.com/game-ci/unity-builder.git');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('targetBuildRepoUrl', () => {
|
||||||
|
it('should not include token in URL when using header auth', () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'header';
|
||||||
|
const url = OrchestratorFolders.targetBuildRepoUrl;
|
||||||
|
expect(url).toBe('https://github.com/myorg/myrepo.git');
|
||||||
|
expect(url).not.toContain('ghp_test123');
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should include token in URL when using url auth (legacy)', () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'url';
|
||||||
|
const url = OrchestratorFolders.targetBuildRepoUrl;
|
||||||
|
expect(url).toBe('https://ghp_test123@github.com/myorg/myrepo.git');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('gitAuthConfigScript', () => {
|
||||||
|
it('should emit http.extraHeader commands in header mode', () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'header';
|
||||||
|
const script = OrchestratorFolders.gitAuthConfigScript;
|
||||||
|
expect(script).toContain('http.extraHeader');
|
||||||
|
expect(script).toContain('GIT_PRIVATE_TOKEN');
|
||||||
|
expect(script).toContain('Authorization: Basic');
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should emit no-op comment in url mode', () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'url';
|
||||||
|
const script = OrchestratorFolders.gitAuthConfigScript;
|
||||||
|
expect(script).toContain('legacy');
|
||||||
|
expect(script).not.toContain('http.extraHeader');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('configureGitAuth', () => {
|
||||||
|
it('should run git config with http.extraHeader in header mode', async () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'header';
|
||||||
|
mockOrchestrator.buildParameters.gitPrivateToken = 'ghp_test123';
|
||||||
|
const { OrchestratorSystem } = require('../services/core/orchestrator-system');
|
||||||
|
|
||||||
|
await OrchestratorFolders.configureGitAuth();
|
||||||
|
|
||||||
|
// Verify the base64 encoding and extraHeader config are correct
|
||||||
|
const expectedEncoded = Buffer.from('x-access-token:ghp_test123').toString('base64');
|
||||||
|
expect(OrchestratorSystem.Run).toHaveBeenCalledWith(expect.stringContaining(expectedEncoded));
|
||||||
|
expect(OrchestratorSystem.Run).toHaveBeenCalledWith(expect.stringContaining('.extraHeader'));
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should not run git config in url mode', async () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'url';
|
||||||
|
const { OrchestratorSystem } = require('../services/core/orchestrator-system');
|
||||||
|
|
||||||
|
await OrchestratorFolders.configureGitAuth();
|
||||||
|
|
||||||
|
expect(OrchestratorSystem.Run).not.toHaveBeenCalled();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should not run git config when no token is available', async () => {
|
||||||
|
mockOrchestrator.buildParameters.gitAuthMode = 'header';
|
||||||
|
mockOrchestrator.buildParameters.gitPrivateToken = '';
|
||||||
|
const originalEnv = process.env.GIT_PRIVATE_TOKEN;
|
||||||
|
delete process.env.GIT_PRIVATE_TOKEN;
|
||||||
|
const { OrchestratorSystem } = require('../services/core/orchestrator-system');
|
||||||
|
|
||||||
|
await OrchestratorFolders.configureGitAuth();
|
||||||
|
|
||||||
|
expect(OrchestratorSystem.Run).not.toHaveBeenCalled();
|
||||||
|
if (originalEnv !== undefined) process.env.GIT_PRIVATE_TOKEN = originalEnv;
|
||||||
|
});
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -1,161 +0,0 @@
|
|||||||
import { OrchestratorFolders } from './orchestrator-folders';
|
|
||||||
|
|
||||||
// Mock Orchestrator
|
|
||||||
jest.mock('../orchestrator', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
buildParameters: {
|
|
||||||
buildGuid: 'test-guid-abc',
|
|
||||||
cacheKey: 'my-cache-key',
|
|
||||||
projectPath: 'test-project',
|
|
||||||
buildPath: 'Builds',
|
|
||||||
maxRetainedWorkspaces: 0,
|
|
||||||
gitPrivateToken: 'ghp_test123',
|
|
||||||
orchestratorRepoName: 'game-ci/unity-builder',
|
|
||||||
githubRepo: 'user/my-game',
|
|
||||||
},
|
|
||||||
lockedWorkspace: '',
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('../../build-parameters', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
shouldUseRetainedWorkspaceMode: jest.fn().mockReturnValue(false),
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('./orchestrator-options', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
useSharedBuilder: false,
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
// Normalize paths for cross-platform test compatibility
|
|
||||||
const normalize = (p: string) => p.replace(/\\/g, '/');
|
|
||||||
|
|
||||||
describe('OrchestratorFolders', () => {
|
|
||||||
describe('static constants', () => {
|
|
||||||
it('repositoryFolder is "repo"', () => {
|
|
||||||
expect(OrchestratorFolders.repositoryFolder).toBe('repo');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('buildVolumeFolder is "data"', () => {
|
|
||||||
expect(OrchestratorFolders.buildVolumeFolder).toBe('data');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('cacheFolder is "cache"', () => {
|
|
||||||
expect(OrchestratorFolders.cacheFolder).toBe('cache');
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('ToLinuxFolder', () => {
|
|
||||||
it('converts backslashes to forward slashes', () => {
|
|
||||||
expect(OrchestratorFolders.ToLinuxFolder('C:\\Users\\test\\project')).toBe('C:/Users/test/project');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('preserves forward slashes', () => {
|
|
||||||
expect(OrchestratorFolders.ToLinuxFolder('/home/user/project')).toBe('/home/user/project');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles mixed slashes', () => {
|
|
||||||
expect(OrchestratorFolders.ToLinuxFolder('some/path\\mixed/slashes\\here')).toBe('some/path/mixed/slashes/here');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles empty string', () => {
|
|
||||||
expect(OrchestratorFolders.ToLinuxFolder('')).toBe('');
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('path computations (non-retained workspace mode)', () => {
|
|
||||||
it('uniqueOrchestratorJobFolderAbsolute uses buildGuid', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.uniqueOrchestratorJobFolderAbsolute);
|
|
||||||
expect(result).toBe('/data/test-guid-abc');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('cacheFolderForAllFull returns /data/cache', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.cacheFolderForAllFull);
|
|
||||||
expect(result).toBe('/data/cache');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('cacheFolderForCacheKeyFull includes cache key', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.cacheFolderForCacheKeyFull);
|
|
||||||
expect(result).toBe('/data/cache/my-cache-key');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('repoPathAbsolute is under job folder', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.repoPathAbsolute);
|
|
||||||
expect(result).toBe('/data/test-guid-abc/repo');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('projectPathAbsolute includes project path', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.projectPathAbsolute);
|
|
||||||
expect(result).toBe('/data/test-guid-abc/repo/test-project');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('libraryFolderAbsolute is under project path', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.libraryFolderAbsolute);
|
|
||||||
expect(result).toBe('/data/test-guid-abc/repo/test-project/Library');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('projectBuildFolderAbsolute uses buildPath', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.projectBuildFolderAbsolute);
|
|
||||||
expect(result).toBe('/data/test-guid-abc/repo/Builds');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('lfsFolderAbsolute is under .git/lfs', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.lfsFolderAbsolute);
|
|
||||||
expect(result).toBe('/data/test-guid-abc/repo/.git/lfs');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('lfsCacheFolderFull is under cache key', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.lfsCacheFolderFull);
|
|
||||||
expect(result).toBe('/data/cache/my-cache-key/lfs');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('libraryCacheFolderFull is under cache key', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.libraryCacheFolderFull);
|
|
||||||
expect(result).toBe('/data/cache/my-cache-key/Library');
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('builderPathAbsolute', () => {
|
|
||||||
it('uses job folder when shared builder is disabled', () => {
|
|
||||||
const result = normalize(OrchestratorFolders.builderPathAbsolute);
|
|
||||||
expect(result).toBe('/data/test-guid-abc/builder');
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('repo URLs', () => {
|
|
||||||
it('unityBuilderRepoUrl includes token and repo name', () => {
|
|
||||||
const url = OrchestratorFolders.unityBuilderRepoUrl;
|
|
||||||
expect(url).toBe('https://ghp_test123@github.com/game-ci/unity-builder.git');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('targetBuildRepoUrl includes token and github repo', () => {
|
|
||||||
const url = OrchestratorFolders.targetBuildRepoUrl;
|
|
||||||
expect(url).toBe('https://ghp_test123@github.com/user/my-game.git');
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('purgeRemoteCaching', () => {
|
|
||||||
it('returns false when env var is not set', () => {
|
|
||||||
const original = process.env.PURGE_REMOTE_BUILDER_CACHE;
|
|
||||||
delete process.env.PURGE_REMOTE_BUILDER_CACHE;
|
|
||||||
expect(OrchestratorFolders.purgeRemoteCaching).toBe(false);
|
|
||||||
if (original !== undefined) process.env.PURGE_REMOTE_BUILDER_CACHE = original;
|
|
||||||
});
|
|
||||||
|
|
||||||
it('returns true when env var is set', () => {
|
|
||||||
const original = process.env.PURGE_REMOTE_BUILDER_CACHE;
|
|
||||||
process.env.PURGE_REMOTE_BUILDER_CACHE = 'true';
|
|
||||||
expect(OrchestratorFolders.purgeRemoteCaching).toBe(true);
|
|
||||||
if (original !== undefined) {
|
|
||||||
process.env.PURGE_REMOTE_BUILDER_CACHE = original;
|
|
||||||
} else {
|
|
||||||
delete process.env.PURGE_REMOTE_BUILDER_CACHE;
|
|
||||||
}
|
|
||||||
});
|
|
||||||
});
|
|
||||||
});
|
|
||||||
@@ -72,14 +72,67 @@ export class OrchestratorFolders {
|
|||||||
return path.join(OrchestratorFolders.cacheFolderForCacheKeyFull, `Library`);
|
return path.join(OrchestratorFolders.cacheFolderForCacheKeyFull, `Library`);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Whether to use http.extraHeader for git authentication (secure, default)
|
||||||
|
* instead of embedding the token in clone URLs (legacy).
|
||||||
|
*/
|
||||||
|
public static get useHeaderAuth(): boolean {
|
||||||
|
return Orchestrator.buildParameters.gitAuthMode !== 'url';
|
||||||
|
}
|
||||||
|
|
||||||
public static get unityBuilderRepoUrl(): string {
|
public static get unityBuilderRepoUrl(): string {
|
||||||
|
if (OrchestratorFolders.useHeaderAuth) {
|
||||||
|
return `https://github.com/${Orchestrator.buildParameters.orchestratorRepoName}.git`;
|
||||||
|
}
|
||||||
|
|
||||||
return `https://${Orchestrator.buildParameters.gitPrivateToken}@github.com/${Orchestrator.buildParameters.orchestratorRepoName}.git`;
|
return `https://${Orchestrator.buildParameters.gitPrivateToken}@github.com/${Orchestrator.buildParameters.orchestratorRepoName}.git`;
|
||||||
}
|
}
|
||||||
|
|
||||||
public static get targetBuildRepoUrl(): string {
|
public static get targetBuildRepoUrl(): string {
|
||||||
|
if (OrchestratorFolders.useHeaderAuth) {
|
||||||
|
return `https://github.com/${Orchestrator.buildParameters.githubRepo}.git`;
|
||||||
|
}
|
||||||
|
|
||||||
return `https://${Orchestrator.buildParameters.gitPrivateToken}@github.com/${Orchestrator.buildParameters.githubRepo}.git`;
|
return `https://${Orchestrator.buildParameters.gitPrivateToken}@github.com/${Orchestrator.buildParameters.githubRepo}.git`;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Shell commands to configure git authentication via http.extraHeader.
|
||||||
|
* Uses GIT_PRIVATE_TOKEN env var so the token never appears in clone URLs or git config output.
|
||||||
|
* This is the same mechanism used by actions/checkout.
|
||||||
|
*
|
||||||
|
* Only emits commands when gitAuthMode is 'header' (default). In 'url' mode,
|
||||||
|
* returns a no-op comment since the token is already in the URL.
|
||||||
|
*/
|
||||||
|
public static get gitAuthConfigScript(): string {
|
||||||
|
if (!OrchestratorFolders.useHeaderAuth) {
|
||||||
|
return `# git auth: using token-in-URL mode (legacy)`;
|
||||||
|
}
|
||||||
|
|
||||||
|
return `# git auth: configuring http.extraHeader (secure mode)
|
||||||
|
if [ -n "$GIT_PRIVATE_TOKEN" ]; then
|
||||||
|
git config --global http.https://github.com/.extraHeader "Authorization: Basic $(printf '%s' "x-access-token:$GIT_PRIVATE_TOKEN" | base64 -w 0)"
|
||||||
|
fi`;
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Configure git authentication via http.extraHeader in the current Node process.
|
||||||
|
* For use in the remote-client where shell scripts aren't used.
|
||||||
|
* Only configures when gitAuthMode is 'header' (default).
|
||||||
|
*/
|
||||||
|
public static async configureGitAuth(): Promise<void> {
|
||||||
|
if (!OrchestratorFolders.useHeaderAuth) return;
|
||||||
|
|
||||||
|
const token = Orchestrator.buildParameters.gitPrivateToken || process.env.GIT_PRIVATE_TOKEN || '';
|
||||||
|
if (!token) return;
|
||||||
|
|
||||||
|
const encoded = Buffer.from(`x-access-token:${token}`).toString('base64');
|
||||||
|
const { OrchestratorSystem } = await import('../services/core/orchestrator-system');
|
||||||
|
await OrchestratorSystem.Run(
|
||||||
|
`git config --global http.https://github.com/.extraHeader "Authorization: Basic ${encoded}"`,
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
public static get buildVolumeFolder() {
|
public static get buildVolumeFolder() {
|
||||||
return 'data';
|
return 'data';
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,53 +0,0 @@
|
|||||||
import OrchestratorNamespace from './orchestrator-guid';
|
|
||||||
|
|
||||||
describe('OrchestratorNamespace', () => {
|
|
||||||
describe('generateGuid', () => {
|
|
||||||
it('generates a guid with correct format', () => {
|
|
||||||
const guid = OrchestratorNamespace.generateGuid('42', 'StandaloneLinux64');
|
|
||||||
// Format: {runNumber}-{platform}-{nanoid4}
|
|
||||||
expect(guid).toMatch(/^42-linux64-[a-z0-9]{4}$/);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('strips "standalone" prefix from platform (case-insensitive)', () => {
|
|
||||||
const guid = OrchestratorNamespace.generateGuid('1', 'StandaloneWindows64');
|
|
||||||
expect(guid).toMatch(/^1-windows64-[a-z0-9]{4}$/);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('lowercases platform name', () => {
|
|
||||||
const guid = OrchestratorNamespace.generateGuid('5', 'Android');
|
|
||||||
expect(guid).toMatch(/^5-android-[a-z0-9]{4}$/);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles numeric run number', () => {
|
|
||||||
const guid = OrchestratorNamespace.generateGuid(100, 'iOS');
|
|
||||||
expect(guid).toMatch(/^100-ios-[a-z0-9]{4}$/);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('generates unique guids on repeated calls', () => {
|
|
||||||
const guids = new Set<string>();
|
|
||||||
for (let i = 0; i < 20; i++) {
|
|
||||||
guids.add(OrchestratorNamespace.generateGuid('1', 'StandaloneLinux64'));
|
|
||||||
}
|
|
||||||
// With 4 alphanumeric chars (36^4 = ~1.7M possibilities), 20 calls should almost certainly be unique
|
|
||||||
expect(guids.size).toBeGreaterThan(1);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles StandaloneOSX platform', () => {
|
|
||||||
const guid = OrchestratorNamespace.generateGuid('7', 'StandaloneOSX');
|
|
||||||
expect(guid).toMatch(/^7-osx-[a-z0-9]{4}$/);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles WebGL platform (no standalone prefix)', () => {
|
|
||||||
const guid = OrchestratorNamespace.generateGuid('3', 'WebGL');
|
|
||||||
expect(guid).toMatch(/^3-webgl-[a-z0-9]{4}$/);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('uses only lowercase alphanumeric characters in nanoid portion', () => {
|
|
||||||
for (let i = 0; i < 10; i++) {
|
|
||||||
const guid = OrchestratorNamespace.generateGuid('1', 'test');
|
|
||||||
const nanoidPart = guid.split('-').pop()!;
|
|
||||||
expect(nanoidPart).toMatch(/^[0-9a-z]{4}$/);
|
|
||||||
}
|
|
||||||
});
|
|
||||||
});
|
|
||||||
});
|
|
||||||
@@ -138,6 +138,10 @@ class OrchestratorOptions {
|
|||||||
return provider || 'local';
|
return provider || 'local';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static get gitAuthMode(): string {
|
||||||
|
return OrchestratorOptions.getInput('gitAuthMode') || 'header';
|
||||||
|
}
|
||||||
|
|
||||||
static get containerCpu(): string {
|
static get containerCpu(): string {
|
||||||
return OrchestratorOptions.getInput('containerCpu') || `1024`;
|
return OrchestratorOptions.getInput('containerCpu') || `1024`;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -302,6 +302,7 @@ export class RemoteClient {
|
|||||||
|
|
||||||
RemoteClientLogger.log(`Initializing source repository for cloning with caching of LFS files`);
|
RemoteClientLogger.log(`Initializing source repository for cloning with caching of LFS files`);
|
||||||
await OrchestratorSystem.Run(`git config --global advice.detachedHead false`);
|
await OrchestratorSystem.Run(`git config --global advice.detachedHead false`);
|
||||||
|
await OrchestratorFolders.configureGitAuth();
|
||||||
RemoteClientLogger.log(`Cloning the repository being built:`);
|
RemoteClientLogger.log(`Cloning the repository being built:`);
|
||||||
await OrchestratorSystem.Run(`git config --global filter.lfs.smudge "git-lfs smudge --skip -- %f"`);
|
await OrchestratorSystem.Run(`git config --global filter.lfs.smudge "git-lfs smudge --skip -- %f"`);
|
||||||
await OrchestratorSystem.Run(`git config --global filter.lfs.process "git-lfs filter-process --skip"`);
|
await OrchestratorSystem.Run(`git config --global filter.lfs.process "git-lfs filter-process --skip"`);
|
||||||
@@ -411,12 +412,7 @@ export class RemoteClient {
|
|||||||
const gitPrivateToken = process.env.GIT_PRIVATE_TOKEN;
|
const gitPrivateToken = process.env.GIT_PRIVATE_TOKEN;
|
||||||
if (gitPrivateToken) {
|
if (gitPrivateToken) {
|
||||||
RemoteClientLogger.log(`Attempting to pull LFS files with GIT_PRIVATE_TOKEN...`);
|
RemoteClientLogger.log(`Attempting to pull LFS files with GIT_PRIVATE_TOKEN...`);
|
||||||
await OrchestratorSystem.Run(`git config --global --unset-all url."https://github.com/".insteadOf || true`);
|
await RemoteClient.configureTokenAuth(gitPrivateToken);
|
||||||
await OrchestratorSystem.Run(`git config --global --unset-all url."ssh://git@github.com/".insteadOf || true`);
|
|
||||||
await OrchestratorSystem.Run(`git config --global --unset-all url."git@github.com".insteadOf || true`);
|
|
||||||
await OrchestratorSystem.Run(
|
|
||||||
`git config --global url."https://${gitPrivateToken}@github.com/".insteadOf "https://github.com/"`,
|
|
||||||
);
|
|
||||||
await OrchestratorSystem.Run(`git lfs pull`, true);
|
await OrchestratorSystem.Run(`git lfs pull`, true);
|
||||||
await OrchestratorSystem.Run(`git lfs checkout || true`, true);
|
await OrchestratorSystem.Run(`git lfs checkout || true`, true);
|
||||||
RemoteClientLogger.log(`Successfully pulled LFS files with GIT_PRIVATE_TOKEN`);
|
RemoteClientLogger.log(`Successfully pulled LFS files with GIT_PRIVATE_TOKEN`);
|
||||||
@@ -432,12 +428,7 @@ export class RemoteClient {
|
|||||||
const githubToken = process.env.GITHUB_TOKEN;
|
const githubToken = process.env.GITHUB_TOKEN;
|
||||||
if (githubToken) {
|
if (githubToken) {
|
||||||
RemoteClientLogger.log(`Attempting to pull LFS files with GITHUB_TOKEN fallback...`);
|
RemoteClientLogger.log(`Attempting to pull LFS files with GITHUB_TOKEN fallback...`);
|
||||||
await OrchestratorSystem.Run(`git config --global --unset-all url."https://github.com/".insteadOf || true`);
|
await RemoteClient.configureTokenAuth(githubToken);
|
||||||
await OrchestratorSystem.Run(`git config --global --unset-all url."ssh://git@github.com/".insteadOf || true`);
|
|
||||||
await OrchestratorSystem.Run(`git config --global --unset-all url."git@github.com".insteadOf || true`);
|
|
||||||
await OrchestratorSystem.Run(
|
|
||||||
`git config --global url."https://${githubToken}@github.com/".insteadOf "https://github.com/"`,
|
|
||||||
);
|
|
||||||
await OrchestratorSystem.Run(`git lfs pull`, true);
|
await OrchestratorSystem.Run(`git lfs pull`, true);
|
||||||
await OrchestratorSystem.Run(`git lfs checkout || true`, true);
|
await OrchestratorSystem.Run(`git lfs checkout || true`, true);
|
||||||
RemoteClientLogger.log(`Successfully pulled LFS files with GITHUB_TOKEN`);
|
RemoteClientLogger.log(`Successfully pulled LFS files with GITHUB_TOKEN`);
|
||||||
@@ -501,4 +492,25 @@ export class RemoteClient {
|
|||||||
|
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Configure git authentication for a token. In header mode (default), uses
|
||||||
|
* http.extraHeader so the token never appears in URLs or git config output.
|
||||||
|
* In url mode (legacy), uses url.insteadOf to embed the token in URLs.
|
||||||
|
*/
|
||||||
|
private static async configureTokenAuth(token: string): Promise<void> {
|
||||||
|
if (OrchestratorFolders.useHeaderAuth) {
|
||||||
|
const encoded = Buffer.from(`x-access-token:${token}`).toString('base64');
|
||||||
|
await OrchestratorSystem.Run(
|
||||||
|
`git config --global http.https://github.com/.extraHeader "Authorization: Basic ${encoded}"`,
|
||||||
|
);
|
||||||
|
} else {
|
||||||
|
await OrchestratorSystem.Run(`git config --global --unset-all url."https://github.com/".insteadOf || true`);
|
||||||
|
await OrchestratorSystem.Run(`git config --global --unset-all url."ssh://git@github.com/".insteadOf || true`);
|
||||||
|
await OrchestratorSystem.Run(`git config --global --unset-all url."git@github.com".insteadOf || true`);
|
||||||
|
await OrchestratorSystem.Run(
|
||||||
|
`git config --global url."https://${token}@github.com/".insteadOf "https://github.com/"`,
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,147 +0,0 @@
|
|||||||
import { FollowLogStreamService } from './follow-log-stream-service';
|
|
||||||
import * as core from '@actions/core';
|
|
||||||
import GitHub from '../../../github';
|
|
||||||
|
|
||||||
// Mock dependencies
|
|
||||||
jest.mock('../../../github', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
updateGitHubCheck: jest.fn(),
|
|
||||||
githubInputEnabled: false,
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('@actions/core', () => ({
|
|
||||||
warning: jest.fn(),
|
|
||||||
setOutput: jest.fn(),
|
|
||||||
setFailed: jest.fn(),
|
|
||||||
error: jest.fn(),
|
|
||||||
getInput: jest.fn().mockReturnValue(''),
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('../../orchestrator', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
buildParameters: {
|
|
||||||
logId: 'test-log-id-123',
|
|
||||||
},
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('../../options/orchestrator-statics', () => ({
|
|
||||||
OrchestratorStatics: {
|
|
||||||
logPrefix: 'TEST',
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('./orchestrator-logger', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
log: jest.fn(),
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
describe('FollowLogStreamService', () => {
|
|
||||||
beforeEach(() => {
|
|
||||||
jest.clearAllMocks();
|
|
||||||
FollowLogStreamService.Reset();
|
|
||||||
FollowLogStreamService.errors = '';
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('Reset', () => {
|
|
||||||
it('resets DidReceiveEndOfTransmission to false', () => {
|
|
||||||
FollowLogStreamService.DidReceiveEndOfTransmission = true;
|
|
||||||
FollowLogStreamService.Reset();
|
|
||||||
expect(FollowLogStreamService.DidReceiveEndOfTransmission).toBe(false);
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('handleIteration', () => {
|
|
||||||
it('detects end of transmission marker', () => {
|
|
||||||
const result = FollowLogStreamService.handleIteration('---test-log-id-123', true, false, '');
|
|
||||||
expect(FollowLogStreamService.DidReceiveEndOfTransmission).toBe(true);
|
|
||||||
expect(result.shouldReadLogs).toBe(false);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('does not trigger end of transmission for non-matching log ID', () => {
|
|
||||||
const result = FollowLogStreamService.handleIteration('---different-log-id', true, false, '');
|
|
||||||
expect(FollowLogStreamService.DidReceiveEndOfTransmission).toBe(false);
|
|
||||||
expect(result.shouldReadLogs).toBe(true);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('detects Library rebuild message', () => {
|
|
||||||
FollowLogStreamService.handleIteration(
|
|
||||||
'Rebuilding Library because the asset database could not be found!',
|
|
||||||
true,
|
|
||||||
false,
|
|
||||||
'',
|
|
||||||
);
|
|
||||||
expect(GitHub.updateGitHubCheck).toHaveBeenCalledWith('Library was not found, importing new Library', '');
|
|
||||||
expect(core.warning).toHaveBeenCalledWith('LIBRARY NOT FOUND!');
|
|
||||||
expect(core.setOutput).toHaveBeenCalledWith('library-found', 'false');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('detects Build succeeded message', () => {
|
|
||||||
FollowLogStreamService.handleIteration('Build succeeded', true, false, '');
|
|
||||||
expect(GitHub.updateGitHubCheck).toHaveBeenCalledWith('Build succeeded', 'Build succeeded');
|
|
||||||
expect(core.setOutput).toHaveBeenCalledWith('build-result', 'success');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('detects Build fail message', () => {
|
|
||||||
FollowLogStreamService.handleIteration('Build fail', true, false, '');
|
|
||||||
expect(GitHub.updateGitHubCheck).toHaveBeenCalled();
|
|
||||||
expect(core.setOutput).toHaveBeenCalledWith('build-result', 'failed');
|
|
||||||
expect(core.setFailed).toHaveBeenCalledWith('unity build failed');
|
|
||||||
expect(core.error).toHaveBeenCalledWith('BUILD FAILED!');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('accumulates error messages with "error " pattern', () => {
|
|
||||||
FollowLogStreamService.handleIteration('error CS0001: Something went wrong', true, false, '');
|
|
||||||
expect(FollowLogStreamService.errors).toContain('error CS0001: Something went wrong');
|
|
||||||
expect(core.error).toHaveBeenCalled();
|
|
||||||
});
|
|
||||||
|
|
||||||
it('accumulates error messages with "error: " pattern', () => {
|
|
||||||
FollowLogStreamService.handleIteration('Fatal Error: Out of memory', true, false, '');
|
|
||||||
expect(FollowLogStreamService.errors).toContain('Fatal Error: Out of memory');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('accumulates "command failed: " messages', () => {
|
|
||||||
FollowLogStreamService.handleIteration('command failed: git pull', true, false, '');
|
|
||||||
expect(FollowLogStreamService.errors).toContain('command failed: git pull');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('accumulates "invalid " messages', () => {
|
|
||||||
FollowLogStreamService.handleIteration('invalid configuration value', true, false, '');
|
|
||||||
expect(FollowLogStreamService.errors).toContain('invalid configuration value');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('accumulates "cannot be found" messages', () => {
|
|
||||||
FollowLogStreamService.handleIteration('Assembly cannot be found', true, false, '');
|
|
||||||
expect(FollowLogStreamService.errors).toContain('Assembly cannot be found');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('appends message to output', () => {
|
|
||||||
const result = FollowLogStreamService.handleIteration('Some normal log line', true, false, 'previous output\n');
|
|
||||||
expect(result.output).toContain('Some normal log line');
|
|
||||||
expect(result.output).toContain('previous output');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('preserves shouldCleanup value', () => {
|
|
||||||
const result = FollowLogStreamService.handleIteration('normal message', true, true, '');
|
|
||||||
expect(result.shouldCleanup).toBe(true);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('does not change shouldReadLogs for normal messages', () => {
|
|
||||||
const result = FollowLogStreamService.handleIteration('Just a regular build log', true, false, '');
|
|
||||||
expect(result.shouldReadLogs).toBe(true);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('includes accumulated errors in Build fail GitHub check message', () => {
|
|
||||||
FollowLogStreamService.errors = '\nprevious error';
|
|
||||||
FollowLogStreamService.handleIteration('Build fail', true, false, '');
|
|
||||||
const updateCall = (GitHub.updateGitHubCheck as jest.Mock).mock.calls[0];
|
|
||||||
expect(updateCall[0]).toContain('previous error');
|
|
||||||
});
|
|
||||||
});
|
|
||||||
});
|
|
||||||
@@ -1,207 +0,0 @@
|
|||||||
import { TaskParameterSerializer } from './task-parameter-serializer';
|
|
||||||
|
|
||||||
// Mock dependencies that TaskParameterSerializer uses internally
|
|
||||||
jest.mock('@actions/core', () => ({
|
|
||||||
getInput: jest.fn().mockReturnValue(''),
|
|
||||||
setOutput: jest.fn(),
|
|
||||||
info: jest.fn(),
|
|
||||||
warning: jest.fn(),
|
|
||||||
error: jest.fn(),
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('../../options/orchestrator-options', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
getInput: jest.fn().mockReturnValue(undefined),
|
|
||||||
ToEnvVarFormat: (input: string) => {
|
|
||||||
if (input.toUpperCase() === input) {
|
|
||||||
return input;
|
|
||||||
}
|
|
||||||
return input
|
|
||||||
.replace(/([A-Z])/g, ' $1')
|
|
||||||
.trim()
|
|
||||||
.toUpperCase()
|
|
||||||
.replace(/ /g, '_');
|
|
||||||
},
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('../../options/orchestrator-options-reader', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
GetProperties: jest.fn().mockReturnValue([]),
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('../../options/orchestrator-query-override', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
queryOverrides: undefined,
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('../hooks/command-hook-service', () => ({
|
|
||||||
CommandHookService: {
|
|
||||||
getHooks: jest.fn().mockReturnValue([]),
|
|
||||||
getSecrets: jest.fn().mockReturnValue([]),
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('../../../input', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {},
|
|
||||||
}));
|
|
||||||
|
|
||||||
jest.mock('../../../github', () => ({
|
|
||||||
__esModule: true,
|
|
||||||
default: {
|
|
||||||
githubInputEnabled: false,
|
|
||||||
},
|
|
||||||
}));
|
|
||||||
|
|
||||||
describe('TaskParameterSerializer', () => {
|
|
||||||
describe('ToEnvVarFormat', () => {
|
|
||||||
it('converts camelCase to UPPER_SNAKE_CASE', () => {
|
|
||||||
expect(TaskParameterSerializer.ToEnvVarFormat('targetPlatform')).toBe('TARGET_PLATFORM');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('converts single word to uppercase', () => {
|
|
||||||
expect(TaskParameterSerializer.ToEnvVarFormat('version')).toBe('VERSION');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('preserves already-uppercase strings', () => {
|
|
||||||
expect(TaskParameterSerializer.ToEnvVarFormat('AWS_REGION')).toBe('AWS_REGION');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles multi-word camelCase', () => {
|
|
||||||
expect(TaskParameterSerializer.ToEnvVarFormat('buildPlatformTarget')).toBe('BUILD_PLATFORM_TARGET');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles string starting with uppercase', () => {
|
|
||||||
expect(TaskParameterSerializer.ToEnvVarFormat('BuildGuid')).toBe('BUILD_GUID');
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('UndoEnvVarFormat', () => {
|
|
||||||
it('converts UPPER_SNAKE_CASE back to camelCase', () => {
|
|
||||||
expect(TaskParameterSerializer.UndoEnvVarFormat('TARGET_PLATFORM')).toBe('targetPlatform');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles single word', () => {
|
|
||||||
expect(TaskParameterSerializer.UndoEnvVarFormat('VERSION')).toBe('version');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles multiple underscores', () => {
|
|
||||||
expect(TaskParameterSerializer.UndoEnvVarFormat('BUILD_PLATFORM_TARGET')).toBe('buildPlatformTarget');
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('round-trip conversion', () => {
|
|
||||||
it('ToEnvVarFormat -> UndoEnvVarFormat returns original for simple camelCase', () => {
|
|
||||||
const original = 'targetPlatform';
|
|
||||||
const envVar = TaskParameterSerializer.ToEnvVarFormat(original);
|
|
||||||
const roundTrip = TaskParameterSerializer.UndoEnvVarFormat(envVar);
|
|
||||||
expect(roundTrip).toBe(original);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('round-trips multi-word keys', () => {
|
|
||||||
const original = 'cacheKey';
|
|
||||||
const envVar = TaskParameterSerializer.ToEnvVarFormat(original);
|
|
||||||
const roundTrip = TaskParameterSerializer.UndoEnvVarFormat(envVar);
|
|
||||||
expect(roundTrip).toBe(original);
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('uniqBy', () => {
|
|
||||||
it('removes duplicates by key function', () => {
|
|
||||||
const items = [
|
|
||||||
{ name: 'A', value: '1' },
|
|
||||||
{ name: 'B', value: '2' },
|
|
||||||
{ name: 'A', value: '3' },
|
|
||||||
];
|
|
||||||
const result = TaskParameterSerializer.uniqBy(items, (x) => x.name);
|
|
||||||
expect(result).toHaveLength(2);
|
|
||||||
expect(result[0].value).toBe('1');
|
|
||||||
expect(result[1].value).toBe('2');
|
|
||||||
});
|
|
||||||
|
|
||||||
it('returns all items when no duplicates', () => {
|
|
||||||
const items = [
|
|
||||||
{ name: 'A', value: '1' },
|
|
||||||
{ name: 'B', value: '2' },
|
|
||||||
{ name: 'C', value: '3' },
|
|
||||||
];
|
|
||||||
const result = TaskParameterSerializer.uniqBy(items, (x) => x.name);
|
|
||||||
expect(result).toHaveLength(3);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('handles empty array', () => {
|
|
||||||
const result = TaskParameterSerializer.uniqBy([], (x) => x.name);
|
|
||||||
expect(result).toHaveLength(0);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('keeps first occurrence when duplicates exist', () => {
|
|
||||||
const items = [
|
|
||||||
{ name: 'KEY', value: 'first' },
|
|
||||||
{ name: 'KEY', value: 'second' },
|
|
||||||
{ name: 'KEY', value: 'third' },
|
|
||||||
];
|
|
||||||
const result = TaskParameterSerializer.uniqBy(items, (x) => x.name);
|
|
||||||
expect(result).toHaveLength(1);
|
|
||||||
expect(result[0].value).toBe('first');
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('blockedParameterNames', () => {
|
|
||||||
it('contains expected blocked names', () => {
|
|
||||||
expect(TaskParameterSerializer.blockedParameterNames.has('0')).toBe(true);
|
|
||||||
expect(TaskParameterSerializer.blockedParameterNames.has('length')).toBe(true);
|
|
||||||
expect(TaskParameterSerializer.blockedParameterNames.has('prototype')).toBe(true);
|
|
||||||
expect(TaskParameterSerializer.blockedParameterNames.has('')).toBe(true);
|
|
||||||
expect(TaskParameterSerializer.blockedParameterNames.has('unityVersion')).toBe(true);
|
|
||||||
expect(TaskParameterSerializer.blockedParameterNames.has('CUSTOM_JOB')).toBe(true);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('does not block valid parameter names', () => {
|
|
||||||
expect(TaskParameterSerializer.blockedParameterNames.has('targetPlatform')).toBe(false);
|
|
||||||
expect(TaskParameterSerializer.blockedParameterNames.has('buildGuid')).toBe(false);
|
|
||||||
expect(TaskParameterSerializer.blockedParameterNames.has('cacheKey')).toBe(false);
|
|
||||||
});
|
|
||||||
});
|
|
||||||
|
|
||||||
describe('readDefaultSecrets', () => {
|
|
||||||
it('returns an array', () => {
|
|
||||||
const secrets = TaskParameterSerializer.readDefaultSecrets();
|
|
||||||
expect(Array.isArray(secrets)).toBe(true);
|
|
||||||
});
|
|
||||||
|
|
||||||
it('includes secrets from environment when present', () => {
|
|
||||||
const originalSerial = process.env.UNITY_SERIAL;
|
|
||||||
process.env.UNITY_SERIAL = 'test-serial';
|
|
||||||
|
|
||||||
const secrets = TaskParameterSerializer.readDefaultSecrets();
|
|
||||||
const serialSecret = secrets.find((s) => s.ParameterKey === 'UNITY_SERIAL');
|
|
||||||
expect(serialSecret).toBeDefined();
|
|
||||||
expect(serialSecret?.ParameterValue).toBe('test-serial');
|
|
||||||
|
|
||||||
if (originalSerial !== undefined) {
|
|
||||||
process.env.UNITY_SERIAL = originalSerial;
|
|
||||||
} else {
|
|
||||||
delete process.env.UNITY_SERIAL;
|
|
||||||
}
|
|
||||||
});
|
|
||||||
|
|
||||||
it('excludes secrets not in environment', () => {
|
|
||||||
const originalSerial = process.env.UNITY_SERIAL;
|
|
||||||
delete process.env.UNITY_SERIAL;
|
|
||||||
|
|
||||||
const secrets = TaskParameterSerializer.readDefaultSecrets();
|
|
||||||
const serialSecret = secrets.find((s) => s.ParameterKey === 'UNITY_SERIAL');
|
|
||||||
expect(serialSecret).toBeUndefined();
|
|
||||||
|
|
||||||
if (originalSerial !== undefined) {
|
|
||||||
process.env.UNITY_SERIAL = originalSerial;
|
|
||||||
}
|
|
||||||
});
|
|
||||||
});
|
|
||||||
});
|
|
||||||
@@ -27,6 +27,7 @@ printenv
|
|||||||
git config --global advice.detachedHead false
|
git config --global advice.detachedHead false
|
||||||
git config --global filter.lfs.smudge "git-lfs smudge --skip -- %f"
|
git config --global filter.lfs.smudge "git-lfs smudge --skip -- %f"
|
||||||
git config --global filter.lfs.process "git-lfs filter-process --skip"
|
git config --global filter.lfs.process "git-lfs filter-process --skip"
|
||||||
|
${OrchestratorFolders.gitAuthConfigScript}
|
||||||
BRANCH="${Orchestrator.buildParameters.orchestratorBranch}"
|
BRANCH="${Orchestrator.buildParameters.orchestratorBranch}"
|
||||||
REPO="${OrchestratorFolders.unityBuilderRepoUrl}"
|
REPO="${OrchestratorFolders.unityBuilderRepoUrl}"
|
||||||
if [ -n "$(git ls-remote --heads "$REPO" "$BRANCH" 2>/dev/null)" ]; then
|
if [ -n "$(git ls-remote --heads "$REPO" "$BRANCH" 2>/dev/null)" ]; then
|
||||||
|
|||||||
@@ -92,6 +92,7 @@ export class BuildAutomationWorkflow implements WorkflowInterface {
|
|||||||
const commands = `mkdir -p ${OrchestratorFolders.ToLinuxFolder(
|
const commands = `mkdir -p ${OrchestratorFolders.ToLinuxFolder(
|
||||||
OrchestratorFolders.builderPathAbsolute,
|
OrchestratorFolders.builderPathAbsolute,
|
||||||
)}
|
)}
|
||||||
|
${OrchestratorFolders.gitAuthConfigScript}
|
||||||
BRANCH="${Orchestrator.buildParameters.orchestratorBranch}"
|
BRANCH="${Orchestrator.buildParameters.orchestratorBranch}"
|
||||||
REPO="${OrchestratorFolders.unityBuilderRepoUrl}"
|
REPO="${OrchestratorFolders.unityBuilderRepoUrl}"
|
||||||
DEST="${OrchestratorFolders.ToLinuxFolder(OrchestratorFolders.builderPathAbsolute)}"
|
DEST="${OrchestratorFolders.ToLinuxFolder(OrchestratorFolders.builderPathAbsolute)}"
|
||||||
|
|||||||
Reference in New Issue
Block a user