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

feat: Tx sign in Keplr #128

Merged
merged 8 commits into from
May 10, 2022
Merged

feat: Tx sign in Keplr #128

merged 8 commits into from
May 10, 2022

Conversation

Senna46
Copy link
Collaborator

@Senna46 Senna46 commented Apr 28, 2022

No description provided.

@Senna46
Copy link
Collaborator Author

Senna46 commented Apr 28, 2022

The signing in Keplr looks work and breaks through the simulation, but errors occur in main transactions.
An error has occur: Error: signature verification failed; please verify account number (127) and chain-id (ununifi-8-private-test): unauthorized
スクリーンショット 2022-04-28 174647

@Senna46
Copy link
Collaborator Author

Senna46 commented Apr 28, 2022

This API response is correct, isn't it?
pub_key field is null.

account: {@type: "/cosmos.auth.v1beta1.BaseAccount", address: "ununifi1hn58ys5xzj8u486azf2ve7fg7jcegrl7hgvj74",…}
@type: "/cosmos.auth.v1beta1.BaseAccount"
account_number: "127"
address: "ununifi1hn58ys5xzj8u486azf2ve7fg7jcegrl7hgvj74"
pub_key: null
sequence: "0"

@kimurayu45z
Copy link
Contributor

The address which haven't announced transactions can't be created the relation with the public key which can be obtained from the signature in transactions, so there is no route to get pubkey from address and it is regular working

@Senna46
Copy link
Collaborator Author

Senna46 commented Apr 28, 2022

OK, Thanks.
So, the problem is the signature....

@Senna46 Senna46 changed the title feat: Tx sign in Keplr [WIP] feat: Tx sign in Keplr May 9, 2022
@Senna46 Senna46 changed the base branch from feature/keplr to develop May 9, 2022 10:00
@Senna46 Senna46 changed the base branch from develop to feature/keplr May 9, 2022 10:01
@Senna46 Senna46 changed the base branch from feature/keplr to develop May 9, 2022 10:03
@Senna46 Senna46 changed the base branch from develop to feature/keplr May 9, 2022 10:03
@Senna46 Senna46 changed the title [WIP] feat: Tx sign in Keplr feat: Tx sign in Keplr May 10, 2022
@Senna46 Senna46 merged commit 7f4cf40 into feature/keplr May 10, 2022
@YasunoriMATSUOKA YasunoriMATSUOKA deleted the feature/keplr-sign branch May 11, 2022 03:59
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

Successfully merging this pull request may close these issues.

2 participants