Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Robo woofy diff tool to check strategy code against diff git commit #3

Open
5 tasks
vany365 opened this issue Jan 6, 2022 · 1 comment
Open
5 tasks

Comments

@vany365
Copy link

vany365 commented Jan 6, 2022

Elevator Pitch:

Robowoofy is a great tool that has eased the operations on doing msig complex transactions and adding new strategies. At the same time it poses an additional risk that anyone that can add a PR to robo woofy could potential attach a malicious strategy or a transferFrom operation if the msig signers are not careful.

The current mitigations are that PRs are restricted to trusted members and we have peer reviews.

an automated tool that runs as a github action would help have an additional layer of security.

Customer Value:

Avoid potential attack vectors in the msig and ensure validity of actions in PR.

Agreed upon Acceptance Criteria (definition of Done)👍

Create a short spec of the tool.

POC of a tool/script that runs on robo woofy PR and alerts if diff of attached strategy code in etherscan matches the linked reviewed commit code and adds a diff checker link for review in the PR.

Task List:

  • Service that grabs the PR and detects if it attaches a strategy. (can detect other operations in contract like transferFrom)
  • Grab the strategy code from etherscan (shows alert if code is not verified)
  • Grab the commit link from github and execute a build and flattening of the strategy file
  • compare build output with etherscan code.
  • Using flattening plugins and tools already built for verification may be useful

Estimation/Estimated Completion date:

a few weeks when task has been started.

@storming0x
Copy link

storming0x commented Jun 21, 2022

i built a cli tool that shows a diff output from a local file, maybe this can be completed using a github action to checkout the repo and run the tool to show the diff ?

https://github.com/storming0x/ethlift-rs

There are still some things to improve it , but currently supports brownie and foundry projects which covers the strategy mixes we use.

command failed, first check your syntax
/robowoofy network=[eth|bsc] fn=[valid_brownie_function_name] send=[true|false] delete-branch-after-send=[true|false]
If that looks good, make sure you have write access on this repository.
Also, make sure you are not running this from a repository fork. Robowoofy must be ran from branch on the repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants