From: Tom Clegg <tom@curoverse.com>
Date: Fri, 9 Oct 2015 18:28:29 +0000 (-0400)
Subject: 7167: Deobfuscate variable names
X-Git-Tag: 1.1.0~1321^2
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/f814b4b5d0c6168c95d2eda87a45c87df07c72c9

7167: Deobfuscate variable names
---

diff --git a/sdk/go/keepclient/perms.go b/sdk/go/keepclient/perms.go
index a73db76bbb..12105c6cfc 100644
--- a/sdk/go/keepclient/perms.go
+++ b/sdk/go/keepclient/perms.go
@@ -76,14 +76,14 @@ func VerifySignature(signedLocator, apiToken string, permissionSecret []byte) er
 		return ErrSignatureMissing
 	}
 	blobHash := matches[1]
-	sigHex := matches[2]
-	expHex := matches[3]
-	if expTime, err := parseHexTimestamp(expHex); err != nil {
+	signatureHex := matches[2]
+	expiryHex := matches[3]
+	if expiryTime, err := parseHexTimestamp(expiryHex); err != nil {
 		return ErrSignatureInvalid
-	} else if expTime.Before(time.Now()) {
+	} else if expiryTime.Before(time.Now()) {
 		return ErrSignatureExpired
 	}
-	if sigHex != makePermSignature(blobHash, apiToken, expHex, permissionSecret) {
+	if signatureHex != makePermSignature(blobHash, apiToken, expiryHex, permissionSecret) {
 		return ErrSignatureInvalid
 	}
 	return nil