services/identity: Close sql connections.
There are two places in the identity server where we were forgetting
to close connections. This resulted in "Too many connections" errors
for some tests.
This change closes connections in those cases.
Change-Id: I3655058ce2026824b43a97d4acfa90b7e01e9c3a
diff --git a/services/identity/internal/revocation/sql_database.go b/services/identity/internal/revocation/sql_database.go
index fb1c409..482c240 100644
--- a/services/identity/internal/revocation/sql_database.go
+++ b/services/identity/internal/revocation/sql_database.go
@@ -38,6 +38,7 @@
func (s *sqlDatabase) IsRevoked(revocationCaveatID []byte) (bool, error) {
rows, err := s.isRevokedStmt.Query(hex.EncodeToString(revocationCaveatID))
+ defer rows.Close()
if err != nil {
return false, err
}
@@ -49,6 +50,7 @@
if err != nil {
return nil, err
}
+ defer rows.Close()
if rows.Next() {
var timestamp time.Time
if err := rows.Scan(×tamp); err != nil {
