scriptoriumadmin/GitNexus
1
0
Fork 0
mirror of https://github.com/abhigyanpatwari/GitNexus.git synced 2026-10-05 02:43:32 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 11

fix(ci-setup): drop double-hash in the ACA script auth comment

Browse source 
authComment lines already begin with `#`, but the injection site prefixed
another `# `, producing `# # Auth: …` in the generated bash script. Inject
the comment block without the extra prefix so each line is single-hashed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Gergo Magyar 2026-06-14 09:21:08 +00:00
parent 642f4419d3
commit 52a73622c7
2 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
gitnexus/src/cli/ci-setup/templates.ts
View file

@ -273,7 +273,7 @@ function buildAcaDeployScript(opts: CiSetupOptions): string {
# Deploy GitNexus as an Azure Container App.
# Requires: az CLI, logged in with Contributor on the target resource group.
#
# ${authComment}
${authComment}
set -euo pipefail

6
gitnexus/test/unit/ci-setup-templates.test.ts
View file

@ -160,6 +160,12 @@ describe('ACA deploy script + connection guidance (U6)', () => {
expect(md).toContain('ingress internal');
expect(md).not.toContain(':4748/api/mcp');
});
it('ACA auth comment is not double-hashed (U10)', () => {
expect(aca({ auth: 'token' })).not.toContain('# # ');
expect(aca({ auth: 'none' })).not.toContain('# # ');
expect(aca({ auth: 'token' })).toContain('# Auth: this deploys with --ingress internal');
});
});
describe('version pinning (U9)', () => {