remove rollback logic when writing reservation fails

libp2p · Jul 6, 2021 · 2a0c38e · 2a0c38e
1 parent c42f8b6
commit 2a0c38e
Showing 1 changed file with 2 additions and 8 deletions.
diff --git a/v2/relay/relay.go b/v2/relay/relay.go
@@ -175,15 +175,9 @@ func (r *Relay) handleReserve(s network.Stream) {
 
 	log.Debugf("reserving relay slot for %s", p)
 
-	err := r.writeResponse(s, pbv2.Status_OK, r.makeReservationMsg(p, expire), r.makeLimitMsg(p))
-	if err != nil {
-		s.Reset()
+	if err := r.writeResponse(s, pbv2.Status_OK, r.makeReservationMsg(p, expire), r.makeLimitMsg(p)); err != nil {
 		log.Debugf("error writing reservation response; retracting reservation for %s", p)
-		r.mx.Lock()
-		delete(r.rsvp, p)
-		r.ipcs.RemoveReservation(p)
-		r.host.ConnManager().UntagPeer(p, "relay-reservation")
-		r.mx.Unlock()
+		s.Reset()
 	}
 }