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

Signers are getting "Invalid owner provided" error #91

Closed
1 task done
joanpan opened this issue Dec 8, 2022 · 9 comments
Closed
1 task done

Signers are getting "Invalid owner provided" error #91

joanpan opened this issue Dec 8, 2022 · 9 comments
Labels
mainnet tx_execution Transaction execution issues

Comments

@joanpan
Copy link
Collaborator

joanpan commented Dec 8, 2022

Issue Category

Transaction execution issue

What happened?

Multiple signers are getting "Invalid owner provided" error. First signer who initiated the transaction did not get the error.
https://dashboard.tenderly.co/public/safe/safe-apps/simulator/30a2ad33-9362-4c3b-94d4-ed224f53a928?trace=0.0.1.2

Transaction was created in the new UI. Tested in the old UI and it seems to working fine. Issue is only in the new UI.

Safe address:
eth:0xb270FE91e8E4b80452fBF1b4704208792A350f53

Date and time this happened or you first noticed this issue

09-12-2022

Network

Ethereum Mainnet

Safe Address

0xb270FE91e8E4b80452fBF1b4704208792A350f53

Token Address

No response

Token Type

None

Transactions

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@github-actions github-actions bot added mainnet tx_execution Transaction execution issues labels Dec 8, 2022
@yagopv
Copy link
Member

yagopv commented Dec 9, 2022

Hey @joanpan , is the transaction in the queue the problematic one?

https://app.safe.global/eth:0xb270FE91e8E4b80452fBF1b4704208792A350f53/transactions/queue

I can (successfully simulate) execute it. The problem is only with the owners not being the first one who signed?

Normally this error can happen with a wrong order in the signatures or with incorrect information signed

@CindyLo0
Copy link
Collaborator

CindyLo0 commented Dec 9, 2022

Hi @yagopv,

We have 3 other users experiencing the same issue.

  1. https://app.safe.global/eth:0x48301Fe520f72994d32eAd72E2B6A8447873CF50/transactions/queue >> version 1.1.1, 5th signer for this one.
  2. https://app.safe.global/eth:0x2cF855E17bD28Bb34982Ca1E4371ef7d3E6817e1/transactions/queue >> This user was able to execute using the old UI. User already updated their safe version but they were on version 1.1.1 at the time the issue was reported.
  3. https://app.safe.global/eth:0xAEe07CFaeA5eaaB0487968AFdcFFceEaCa48851E/transactions/queue >> version 1.1.1, issue is with the currently queued transaction.

@yagopv
Copy link
Member

yagopv commented Dec 9, 2022

so they are all in 1.1.1 right?

@CindyLo0
Copy link
Collaborator

CindyLo0 commented Dec 9, 2022

so they are all in 1.1.1 right?

That is correct.

@CindyLo0
Copy link
Collaborator

CindyLo0 commented Dec 9, 2022

For user 3 with safe address https://app.safe.global/eth:0xAEe07CFaeA5eaaB0487968AFdcFFceEaCa48851E/transactions/queue, this is the signer address used 0x2aC29585a7A1371A83e1C0818F4c45A92DDf8Db2 when error came up.

Tenderly - https://dashboard.tenderly.co/public/safe/safe-apps/simulator/713435ea-e699-4cf6-9cc1-3e0d7c72b2c4

@yagopv
Copy link
Member

yagopv commented Dec 9, 2022

The transactions with one missing signer i guess they tried to sign and execute right? For scoping the error to the execution

@CindyLo0
Copy link
Collaborator

CindyLo0 commented Dec 9, 2022

The transactions with one missing signer i guess they tried to sign and execute right? For scoping the error to the execution

That is correct. 5th owner was a typo, I meant to say 6th signer.

@yagopv
Copy link
Member

yagopv commented Dec 9, 2022

Opened a ticket in the web-core repo (new UI)

@JagoFigueroa
Copy link

Hi! The fix for this ticket was included on our last release of the web interface so I will go ahead and close it.

Cheers! (and sorry for the spam today, going through every ticket on the repo)

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

No branches or pull requests

4 participants