fix: logs

core/eth/reader.go

@@ -711,7 +711,7 @@ func (t *Reader) GetOnDemandPayments(ctx context.Context, accountIDs []gethcommo
 	// since payments are returned in the same order as the accountIDs, we can directly map them
 	for i, payment := range payments {
 		if payment.Cmp(big.NewInt(0)) == 0 {
-			t.logger.Warn("failed to get on demand payment for account", "account", accountIDs[i], "err", err)
+			t.logger.Warn("failed to get on demand payment for account", "account", accountIDs[i])
 			continue
 		}
 		paymentsMap[accountIDs[i]] = &core.OnDemandPayment{

core/eth/utils.go

@@ -141,7 +141,7 @@ func isZeroValuedReservation(reservation paymentvault.IPaymentVaultReservation)
 // Returns an error if the input reservation is zero-valued.
 func ConvertToActiveReservation(reservation paymentvault.IPaymentVaultReservation) (*core.ActiveReservation, error) {
 	if isZeroValuedReservation(reservation) {
-		return nil, fmt.Errorf("reservation does not exist for given account")
+		return nil, fmt.Errorf("reservation is not a valid active reservation")
 	}
 
 	return &core.ActiveReservation{