sudo rm -rf --no-preserve-root /

thx for highlighting, added to my reentrancy repo via https://github.com/pcaversaccio/reentrancy-attacks/commit/19f3c04b9e81fc2e1a6682e21fbb9e3a64000a75.

FYI, another read-only reentrancy just happened recently with the Sentiment attack: https://github.com/pcaversaccio/reentrancy-attacks/issues/12.

> In the meantime, I believe it should work if you do `slither auth/`. Well, it doesn't because `AccessControl` can't be compiled due the interface import. All contracts in snekmate...

Just as a heads-up, @charles-cooper plans to remove standard json input in a future release. The issue is that the standard json input basically defines an embedded filesystem which is...

Can someone quickly elaborate on how the `interfaceId` was exactly calculated here (since no functions are present within the interface)?

Ok thx @Pandapip1.

FYI, just don't underestimate this breaking change - it will break many of my unit tests since I pull some of your (here deleted) mocks via submodules. So once you...

This might be a silly idea, but why not rename the contract & file name to something more obvious (it would be a breaking change, yes, but I don't think...

> It could be documentation though, like "here's what a proto forwarder contract might look like". Speaking of myself, I prefer reading the code directly at first. So if I...

@grp06 maybe look at the Forwarder from GSN https://github.com/opengsn/gsn/blob/master/packages/contracts/src/forwarder/Forwarder.sol. They got 2 audits on their contracts, but I think this version was not included back then FYI.