Failure to delete sessions should be non-fatal

99designs · Dec 28, 2017 · 9bc62f5 · 9bc62f5
1 parent d34a2cf
commit 9bc62f5
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/vault/sessions.go b/vault/sessions.go
@@ -73,9 +73,9 @@ func (s *KeyringSessions) Sessions() ([]KeyringSession, error) {
 		if IsSessionKey(k) {
 			ks, _ := parseKeyringSession(k, s.Config)
 			if ks.IsExpired() {
-				log.Printf("Session %s is expired, deleting", k)
+				log.Printf("Session %s is expired, attempting deleting", k)
 				if err := s.Keyring.Remove(k); err != nil {
-					return nil, err
+					log.Printf("Error deleting session: %v", err)
 				}
 				continue
 			}