From defb39f8d40abc272b6a439f46ec149032beb1ab Mon Sep 17 00:00:00 2001 From: James Phillips Date: Thu, 4 Aug 2016 07:46:59 -0700 Subject: [PATCH] Removes a TODO comment. 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. --- consul/acl.go | 5 ----- 1 file changed, 5 deletions(-) diff --git a/consul/acl.go b/consul/acl.go index 7a472134f..a4da8211c 100644 --- a/consul/acl.go +++ b/consul/acl.go @@ -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) }