letsencrypt
/
boulder
mirror of https://github.com/letsencrypt/boulder.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix review comments.

This commit is contained in:
Jacob Hoffman-Andrews 2015-10-06 16:12:23 -07:00
parent e0cdd13bbb
commit 8af3bd953b
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ra/registration-authority.go
View File

@ -515,7 +515,7 @@ func (ra *RegistrationAuthorityImpl) checkCertificatesPerNameLimit(names []strin
// Shouldn't happen, but let's be careful anyhow.
return errors.New("StorageAuthority failed to return a count for every name")
}
if count >= int(limit.GetThreshold(name)) {
if int64(count) >= limit.GetThreshold(name) {
badNames = append(badNames, name)
}
}
@ -529,7 +529,6 @@ func (ra *RegistrationAuthorityImpl) checkCertificatesPerNameLimit(names []strin
func (ra *RegistrationAuthorityImpl) checkLimits(names []string) error {
limits := ra.rlPolicies
// Check total certificate rate limiting
if limits.TotalCertificates.Enabled() {
totalIssued, err := ra.getIssuanceCount()
if err != nil {
@ -539,7 +538,6 @@ func (ra *RegistrationAuthorityImpl) checkLimits(names []string) error {
return core.RateLimitedError("Certificate issuance limit reached")
}
}
// Check rate limiting by name
if limits.CertificatesPerName.Enabled() {
err := ra.checkCertificatesPerNameLimit(names, limits.CertificatesPerName)
if err != nil {