Summary

encoding multiple dynamic values using abi.encodepacked raises alarms in slither, as well as being highly unrecommended in the solidity docs

Vulnerability Details

could result in hash collisions

Impact

--

Tools Used

slither static analyzer

Recommendations

make the function internal instead of public, as well as using abi.encode instead of abi.encodepacked for multiple dynamic variables as recommended in the official solidity documentation