Multiple Solidity pragma in contracts/mocks/DummyRouter.sol

[code423n4]

Handle

fatima_naz

Vulnerability details

Impact

In contracts/mocks/DummyRouter.sol
pragma solidity ^0.8.3;
It is better to use one Solidity compiler version across all contracts instead of different versions with different bugs and security checks. All the other files use 0.8.9 .

Proof of Concept

https://github.com/crytic/slither/wiki/Detector-Documentation#different-pragma-directives-are-used

Tools Used

Recommended Mitigation Steps

change the line with
pragma solidity 0.8.9;

[adrien-supizet]

This is one issue described in multiple PRs #19 #20 #21 #22 #23 #24
We don't believe this is an issue at all as the contracts deployed on the blockchain sometimes have older pragma, we believe it's normal to use mock contracts with the same pragma when we copy it from ehterscan.

[alcueca]

One issue about unlocked pragmas is enough.