agent: fix panic when logging about protocol version config use. (#12962

) The log line comes before the agent logger has been setup, therefore we need to use the UI logging to avoid panic.
hashicorp · May 19, 2022 · e79bc2e · e79bc2e
1 parent 3c71ff1
commit e79bc2e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.changelog/12962.txt b/.changelog/12962.txt
@@ -0,0 +1,3 @@
+```release-note:bug
+agent: fixed a panic on startup when the `server.protocol_version` config parameter was set
+```
diff --git a/command/agent/command.go b/command/agent/command.go
@@ -441,7 +441,7 @@ func (c *Command) IsValidConfig(config, cmdConfig *Config) bool {
 	// ProtocolVersion has never been used. Warn if it is set as someone
 	// has probably made a mistake.
 	if config.Server.ProtocolVersion != 0 {
-		c.agent.logger.Warn("Please remove deprecated protocol_version field from config.")
+		c.Ui.Warn("Please remove deprecated protocol_version field from config.")
 	}
 
 	return true