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

Upgrade Token contract to include a rescueERC20 function #1064

Closed
BlurayDisc opened this issue Apr 10, 2022 · 1 comment
Closed

Upgrade Token contract to include a rescueERC20 function #1064

BlurayDisc opened this issue Apr 10, 2022 · 1 comment

Comments

@BlurayDisc
Copy link

BlurayDisc commented Apr 10, 2022

Hi wormhole dev team,

Purpose

Include a rescueERC20 function in the token's contract in order to rescue existing customer stuck funds and take wormhole's customer oriented attitude to the next level!
https://github.com/certusone/wormhole/blob/dev.v2/ethereum/contracts/bridge/token/TokenImplementation.sol

Background

This opportunity comes from a customer issue of my own...
I've been in contact with your customer support team for about 5 month, for some fund that i accident sent to this contract: Token UST (Wormhole).

I was acknowledged the team has been stretched quite thin at the moment so i don't have any expectations for a time frame for when this will be implemented, which is totally fine.

Implementation

From what i understand the wormhole contracts are openzepplin proxy based / upgradable contract, which would make this possible.

What i think the steps are:

  1. Implement a rescueERC20 function that is similar to what's in the Token USD Coin (PoS) contract:

USDC Rescuable interface:
https://github.com/centrehq/centre-tokens/blob/master/contracts/v1.1/Rescuable.sol
Blockchain explorer function call screenshot:
image

  1. Upgrade the current UST contract on wormhole and assign rescuer to a internal team's address.
  2. Perform the rescue function call.

Expected Result

  • Implement Rescuable.
  • I don't have an expectation or date for step 2 or 3, as the fund has already been miss placed / stuck for 5 month and i totally wouldn't mind to wait a bit longer...
  • I would really appreciate however for any early actions the team has taken.
  • For early actions taken i'm willing to forfit a part of the fund being stuck and use it as a bounty for the developer's time and assitance.

At last, thanks and looking forward to the developer's response!

@BlurayDisc BlurayDisc changed the title Upgrade Token contracts to include a rescueERC20 function Upgrade Token contract to include a rescueERC20 function Apr 10, 2022
@BlurayDisc BlurayDisc changed the title Upgrade Token contract to include a rescueERC20 function [BOUNTY $800 USDC!!] Upgrade Token contract to include a rescueERC20 function May 13, 2022
@claudijd
Copy link
Contributor

Thanks for your report. We understand the unfortunate situation you and your customer are in. However, the proposal would violate the decentralized nature of Wormhole and is something we cannot accomodate.

@claudijd claudijd changed the title [BOUNTY $800 USDC!!] Upgrade Token contract to include a rescueERC20 function Upgrade Token contract to include a rescueERC20 function May 13, 2022
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