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.
pull/2237/head
James Phillips 8 years ago
parent 096d5ff3bb
commit f44bc7e97a
No known key found for this signature in database
GPG Key ID: 77183E682AC5FC11

@ -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)
}

Loading…
Cancel
Save