From 966b3a7ebe0b6736ca70d18be8b44b37ead68473 Mon Sep 17 00:00:00 2001 From: rcourtman Date: Mon, 13 Oct 2025 19:18:21 +0000 Subject: [PATCH] fix: Setup script fmt.Sprintf argument mismatch causing bash syntax error Fixed a fmt.Sprintf argument alignment issue in the PVE setup script that caused a bash syntax error at the end of script execution. The error manifested as "syntax error near unexpected token EXTRA" followed by the serverHost URL. Root cause: 23 arguments were provided for 22 %s placeholders. An extra tokenName at position 15 pushed all subsequent arguments off by one, leaving the final serverHost with no placeholder to fill. Fix: Removed duplicate tokenName at position 15 and ensured serverHost is correctly positioned at position 22 for the "Host URL" placeholder. --- internal/api/config_handlers.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/api/config_handlers.go b/internal/api/config_handlers.go index df7b08b..0168a0f 100644 --- a/internal/api/config_handlers.go +++ b/internal/api/config_handlers.go @@ -3884,7 +3884,7 @@ if [ "$AUTO_REG_SUCCESS" != true ]; then fi `, serverName, time.Now().Format("2006-01-02 15:04:05"), pulseIP, tokenName, tokenName, tokenName, tokenName, tokenName, tokenName, - authToken, pulseURL, serverHost, tokenName, tokenName, tokenName, storagePerms, pulseURL, sshPublicKey, pulseURL, authToken, pulseURL, tokenName, serverHost) + authToken, pulseURL, serverHost, tokenName, tokenName, storagePerms, pulseURL, sshPublicKey, pulseURL, authToken, pulseURL, tokenName, serverHost) } else { // PBS script = fmt.Sprintf(`#!/bin/bash