Golint fixes to builds work

core/objects.go

@@ -97,10 +97,10 @@ const (
 	ChallengeTypeDNS01      = "dns-01"
 )
 
-// The suffix appended to pseudo-domain names in DVSNI challenges
+// TLSSNISuffix is appended to pseudo-domain names in DVSNI challenges
 const TLSSNISuffix = "acme.invalid"
 
-// The label attached to DNS names in DNS challenges
+// DNSPrefix is attached to DNS names in DNS challenges
 const DNSPrefix = "_acme-challenge"
 
 func (pd *ProblemDetails) Error() string {

policy/policy-authority.go

@@ -98,7 +98,7 @@ type SyntaxError struct{}
 // Internet.
 type NonPublicError struct{}
 
-// TLDError indicates that one or more identifiers was an ICANN-managed TLD
+// ErrICANNTLD indicates that one or more identifiers was an ICANN-managed TLD
 var ErrICANNTLD = errors.New("Name is an ICANN TLD")
 
 // ErrBlacklisted indicates we have blacklisted one or more of these

ra/registration-authority.go

@@ -26,9 +26,9 @@ import (
 	blog "github.com/letsencrypt/boulder/log"
 )
 
-// 10 month default authorization lifetime. When used with a 90-day cert
+// DefaultAuthorizationLifetime is the 10 month default authorization lifetime.
-// lifetime, this allows creation of certs that will cover a whole year,
+// When used with a 90-day cert lifetime, this allows creation of certs that will
-// plus a grace period of a month.
+// cover a whole year, plus a grace period of a month.
 // TODO(jsha): Read from a config file.
 const DefaultAuthorizationLifetime = 300 * 24 * time.Hour
 

va/validation-authority.go

@@ -36,7 +36,7 @@ const maxRedirect = 10
 
 var validationTimeout = time.Second * 5
 
-// Returned by CheckCAARecords if it has to follow too many
+// ErrTooManyCNAME is returned by CheckCAARecords if it has to follow too many
 // consecutive CNAME lookups.
 var ErrTooManyCNAME = errors.New("too many CNAME/DNAME lookups")