Beedle - Oracle free perpetual lending

Summary

In Fees.sol we use swapRouter address which is hardcoded:

The problem is that this address may not be valid for all chains that will be deployed.

Vulnerability Details

From the discord server, we have information that the protocol will be debugged at different times.

Hardcoding the address of ISwapRouter can lead to issues, especially when deploying the contract across different chains. Different chains may have different addresses and hardcoding an address may limit the contract's portability.

Is better to provide the address as a constructor parameter. This way, when the contract is deployed, the address of the dependency can be specified, allowing the contract to be used across different chains.

Impact

swapRouter maybe not work on some chains

Tools Used

Visual Studio Code

Recommendations

Set swapRouter address in the constructor on every chain.