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

Removes a TODO comment.

Browse source 
Decided we don't need to log anything about the token here. If the
token is not valid then the client will get an error about that, so
anything that can happen here is related to talking to the server in
the ACL datacenter, so not specific to the token.
This commit is contained in:
James Phillips 2016-08-04 07:46:59 -07:00
parent 93a7fd0561
commit defb39f8d4
No known key found for this signature in database
GPG key ID: 77183E682AC5FC11
1 changed files with 0 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
consul/acl.go
View file

@ -194,11 +194,6 @@ func (c *aclCache) lookupACL(id, authDC string) (acl.ACL, error) {
if strings.Contains(err.Error(), aclNotFound) {
return nil, errors.New(aclNotFound)
} else {
// TODO (slackpad) - We used to print a few characters of the
// token here if the token was long enough. This was bugging me
// so I deleted it. We should probably print a hash of the token,
// or better yet let's add another ID to tokens to identify them
// without giving away their privileges.
c.logger.Printf("[ERR] consul.acl: Failed to get policy from ACL datacenter: %v", err)
}