luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

Remove log line about server mgmt token init (#15610) 

* Remove log line about server mgmt token init

Currently the server management token is only being bootstrapped in the
primary datacenter. That means that servers on the secondary datacenter
will never have this token available, and would log this line any time a
token is resolved.

Bootstrapping the token in secondary datacenters will be done in a
follow-up.

* Add changelog entry
This commit is contained in:
Freddy 2022-11-29 15:56:03 -07:00 committed by GitHub
parent 71f7f2e3dc
commit 7641d10184
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
.changelog/15610.txt Normal file
View File

@ -0,0 +1,3 @@
```release-note:bug
acl: avoid debug log spam in secondary datacenter servers due to management token not being initialized.
```

1
agent/consul/acl_server.go
View File

@ -116,7 +116,6 @@ func (s *serverACLResolverBackend) IsServerManagementToken(token string) bool {
return false return false
} }
if mgmt == "" { if mgmt == "" {
s.logger.Debug("server management token has not been initialized")
return false return false
} }
return subtle.ConstantTimeCompare([]byte(mgmt), []byte(token)) == 1 return subtle.ConstantTimeCompare([]byte(mgmt), []byte(token)) == 1