[N-1] Event is missing indexed fields

[juniormp]

Comments

If you're creating an event function with more than two parameters, it's a good idea to index the ones that you'll be searching for later. This will help you filter through the logs and find the data you need more easily. Just remember that indexing too many fields can increase the gas cost, so be mindful of that as well.

Instances

event BathTokenCreated(address bathToken, address underlying);
event BuddySpawned(address bathToken, address bathBuddy);
https://github.com/juniormp/audit-2023-04-rubicon/blob/511636d889742296a54392875a35e4c0c4727bb7/contracts/BathHouseV2.sol#L23
https://github.com/juniormp/audit-2023-04-rubicon/blob/511636d889742296a54392875a35e4c0c4727bb7/contracts/BathHouseV2.sol#L24