<!-- canonical: https://0xsimao.com/findings/nftperp-i-include-relevant-easier-debug -->

# Errors could include relevant arguments whenever possible, making it easier to debug

Low/Info · Three Sigma · NFT perpetuals · 2nd January, 2024

Finding 3S-NFTPerp-N08 of the Nftperp Exchange security review.

- Report: /reports/nftperp-i
- Source: https://cdn.sanity.io/files/qoqld077/production/c19530de75e234ad15694b4563edb1fc9d2a3fd8.pdf

---

### Description

While during testing it's possible to get the relevant information using other methods
(console.log comes to mind), when the contract is deployed and transactions are live, it's
harder to debug without the information being contained in the error.
This could be even more relevant in case of a suspiscious transaction that is being
analyzed, where time is very important to prevent damage.
See [`ClearingHouse`](https://github.com/nftperp/NFTPerp-V2-Contracts/blob/1c16f2010a851ac23aec73822d55388901bbe5e7/src/ClearingHouse.sol#L78-L90), for example.

### Status

Acknowledged
