Omniscia Bluejay Finance Audit

WhitelistSalePrivate Static Analysis Findings

WhitelistSalePrivate Static Analysis Findings

PAC-01S: Inexistent Event Emissions

Description:

The linked functions adjust sensitive contract variables yet do not emit an event for them.

Example:

packages/contracts/contracts/periphery/WhitelistSalePrivate.sol
144function setPause(bool _paused) public onlyOwner {
145 paused = _paused;
146}
147
148function setMerkleRoot(bytes32 _merkleRoot) public onlyOwner {
149 _setMerkleRoot(_merkleRoot);
150}
151
152function setPrice(uint256 _price) public onlyOwner {
153 price = _price;
154}
155
156function setVestingStart(uint256 _vestingStart) public onlyOwner {
157 vestingStart = _vestingStart;
158}
159
160function setVestingPeriod(uint256 _vestingPeriod) public onlyOwner {
161 vestingPeriod = _vestingPeriod;
162}

Recommendation:

We advise an event to be coded and correspondingly emitted to ensure off-chain processes can properly react to these changes.

Alleviation:

Events and corresponding event emissions have been introduced throughout the relevant setter functions as advised.