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

Remove RRP-specific withdrawal implementation from Airnode v1 #1877

Closed
bbenligiray opened this issue Sep 8, 2023 · 4 comments
Closed

Remove RRP-specific withdrawal implementation from Airnode v1 #1877

bbenligiray opened this issue Sep 8, 2023 · 4 comments

Comments

@bbenligiray
Copy link
Member

We should first research if/how often the RRP withdrawal functionality is used on the public chains. Assuming that it's little to none, I propose to remove the withdrawal functionality from Airnode.

Related to #1558

@Ashar2shahid
Copy link
Contributor

Ashar2shahid commented Sep 9, 2023

User's often use RrpRequesterV0 which makes the contract itself the sponsor. The previous guides/tutorials did not have a withdrawal section and user's funds were locked in the sponsorWallets because the contracts had no withdrawal functions. Since then we added a "Withdrawal Section" in airnode/qrng . Arcadium's sponsorWallet has 0.688 eth locked in their sponsorWallet because the QRNG guides at the time didn't go through withdrawals and the user overestimated how much they should charge for each request. QRNG users also request info about withdrawals (but not frequently).

@dcroote
Copy link
Contributor

dcroote commented Sep 10, 2023

QRNG users also request info about withdrawals (but not frequently).

For example here is a PR clarifying the lack of withdrawal functionality early on due to a Discord discussion: api3dao/old-api3-docs#1107

I use withdrawals in nearly every release test (e.g. #1819, #1839, #1842), and like the functionality's existence so that I don't repeatedly leave testnet funds in various sponsor wallets. That said, having withdrawals external to RRP should also be fine and I envision we can trim a lot from the docs / Airnode repo.

@bbenligiray
Copy link
Member Author

Thanks for your input. After thinking a bit more, I realize that simply removing the withdrawal functionality from Airnode in favor of something like #1558 (comment) will still be breaking, because people will have set sponsors to be contracts and contracts can't sign messages. They can use ERC1271 but that functionality needs to be built in from the start. So it feels like this is too breaking of a change for v0, but we can consider it for v1 (so I edited the issue title).

@bbenligiray bbenligiray changed the title Remove RRP-specific withdrawal implementation from Airnode Remove RRP-specific withdrawal implementation from Airnode v1 Sep 20, 2023
@bbenligiray
Copy link
Member Author

In fact, the only thing to do about this issue at the moment (in the absence of an Airnode v1) is to update this contract to enable #1558 instead. Feel free to re-open the issue if you still think that we should remove withdrawals from v0.

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

3 participants