safe-global / safe-apps-list

MIT License
20 stars 31 forks source link

Add Kiln app #323

Closed loicttn closed 10 months ago

loicttn commented 11 months ago

Revision checks

Name/Description

Kiln Safe dApp enables Safe users to stake ETH on dedicated validators by multiple of 32 ETH.

The Staking Smart contract manages the deposit of each 32 ETH deposited to dedicated validators registered by the Operator (here Kiln) on the smart contract.

The contract deterministically creates 2 recipient addresses for Execution Layer and Consensus Layer rewards per validator deposited on which validator rewards will be received. Validator owner (wallet that deposited) can then withdraw these rewards, net of the 8% commission currently set on the contract.

This staking smart contract has been deployed for multiple integrators, like ledger live that reached +$1b deposited value, and was audited by Spearbit, Halborn, Ledger Donjon and multiple customer security team. A bug bounty is also live on immunefi. More info on security here.

We also end to end tested the Kiln safe app by staking, request validator exit ad withdrawing rewards + collateral unstaking 32 ETH.

Type

Compatible Networks

- Mainnet

Audit document

See here.

Code for review

https://github.com/kilnfi/staking-dapp (private mirror for this review, can you share the github handles we need to invite? Thanks!)

IPFS hash/App URL

https://safe-app.kiln.fi

Team information

- Project logo: https://safe-app.kiln.fi/favicon.ico
- Project landing page: https://kiln.fi/
- Team or discord contact: hello (at] kiln .fi
- Twitter link: @Kiln_finance
- Github link: https://github.com/kilnfi/staking-dapp
kirkkonen commented 10 months ago

Hi @loicttn ! Sorry for the delayed response. Safe{Wallet} team here. Judging by the staking contract tx data on Etherscan, you got 3 active depositors during the last month, is that accurate?

loicttn commented 10 months ago

Hey @kirkkonen on this instance of the contract yes and one of them was an E2E test from our end to validate the safe dapp flow (see links in my message above).

Note that this smart contract is made to be white-labelled and has already been deployed for multiple integrators where each integrator has its own deployed instance. For example Ledger Live instance reached $1b+ deposited.

This instance with 3 stakes is just dedicated to Kiln direct integrations that we want to integrate on Safe here.

kirkkonen commented 10 months ago

Gotcha thanks @loicttn ! Moving this one to the technical assessment stage then.

katspaugh commented 10 months ago

@loicttn could you invite @katspaugh to the repo (readonly)?

loicttn commented 10 months ago

@loicttn could you invite @katspaugh to the repo (readonly)?

You should have received an email to access https://github.com/kilnfi/staking-dapp :)

katspaugh commented 10 months ago

Thank you, enabled. ✅

https://app.safe.global/share/safe-app?appUrl=https%3A%2F%2Fsafe-app.kiln.fi&chain=eth

Please test on your side if it's working as expected. I can be now removed from the github repo.