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: imkey cosmos support multi-network function #133

Open
wants to merge 18 commits into
base: dev
Choose a base branch
from

Conversation

xiaoguang1010
Copy link
Contributor

Summary of Changes

imkey cosmos support multi-network function

Motivation and Context

How Has This Been Tested? (Test Plan)

Other information

Screenshots (if appropriate):

Final checklist

  • Did you test both iOS and Android(if applicable)?
  • Is a security review needed(consenlabs/security)?

Security checklist (only for leader check)

  • No backdoor risk
    • Check for unknown network request urls, and script/shell files with unclear purposes,
    • The backend service cannot expose leaked data interfaces for various reasons (even for testing purposes)
  • No network communication protocol risk
    • Check whether to introduce unsafe network calls such as http/ws
  • No import potentially risk 3rd library
    • Check whether 3rd dependent library is import
    • Don't use an unknown third-party library
    • Check the 3rd library sources are fetched from normal sources, such as npm, gomodule, maven, cocoapod, Do not use unknown sources
    • Check github Dependabot alerts, Whether to add new issues
  • Private data not exposed
    • Check whether there are exclusive ApiKey, privatekey and other private information uploaded to git
    • Check if the packaged keystore has been uploaded to git

tyrone98 and others added 18 commits July 31, 2024 16:48
* add dogecoin testnet & mainnet

* add dogecoin extend public key tests
* feat: add hrp to coininfo

* feat: add stride address test
* add dogecoin testnet & mainnet

* add dogecoin extend public key tests

* fix: fix derive dogecoin unsupported_chain error

---------

Co-authored-by: xiaoguang <[email protected]>
* feat: add p2wpkhp2tr address generation

* feat: add bitcoin Mixed signature function

* test: modify p2wpkh test case

* feat: add bitcoin p2wpkh sign

* feat: add test case and add p2tr sign

* test: add test case

* test: modify btc test case

* feat: code optimization

* feat: modify display_addres and get_address

* feat: add bitcoin p2tr transaction

* test: add p2tr test case

* test: add bitcoin transaction sign function test

* feat: code optimization

* feat: pass in the tweaked public key when signing

* chore: code format

* feat: add imKey PSBT feature code

* feat: add p2wpkhp2tr address generation

* feat: add bitcoin Mixed signature function

* test: modify p2wpkh test case

* feat: add bitcoin p2wpkh sign

* feat: add test case and add p2tr sign

* test: add test case

* test: modify btc test case

* feat: code optimization

* feat: modify display_addres and get_address

* feat: add bitcoin p2tr transaction

* test: add p2tr test case

* test: add bitcoin transaction sign function test

* feat: code optimization

* feat: pass in the tweaked public key when signing

* chore: code format

* chore: switch to staging env

* feat: add imKey PSBT feature code

* feat: derive_account and derive_sub_account support native segwit address and bech32 address

* feat: add bitcoin psbt sign transaction

* feat: add bitcoin p2tr script sign

* fix: fix psbt legacy transaction bug

* chore: psbt code optimized

* fix: fix preview info calc error bug

* fix: modify change path to account path

* chore: code optimized

* feat: psbt transaction support sign message

* feat: add bip-322 sign message feature

* feat: add bitcoin sign message api

* chore: remove address verify function

* chore: get_utxo_pub_key function removes the network param

* chore: modify tcx version to 2.8.0

* chore: bip322 sign message  add path check and code optimization

* build: moify ios CI run-on version to macos-14

* fix: fix ios CI build error

* fix: fix cargo check error

* chore: optimization imKey public key conversion

* chore: PubKeyParam removes isSegwit field
* ci: run unittest when push

* ci: rename workflow name

* ci: fix make command error

* fix: missing test data
Bumps [openssl](https://github.com/sfackler/rust-openssl) from 0.10.62 to 0.10.66.
- [Release notes](https://github.com/sfackler/rust-openssl/releases)
- [Commits](sfackler/rust-openssl@openssl-v0.10.62...openssl-v0.10.66)

---
updated-dependencies:
- dependency-name: openssl
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: force using known hrp

* feat: cosmos derive account by chainId
…[R2D2-12387] (#128)

* fix: fixed the issue of Legacy and Nested Segwit transaction failure when btc not upgraded

* fix: wrong to address cannot be traded when appelt is not upgraded

* chore: optimization error message prompts
* feat: tron sign message support v2 version

* chore: tron sign message v2 review problem optimization

* chore: add proro field description
* feat: support dogecoin address

* feat: ikc supprot dogecoin

* test: add dogecoin device_accounts test case

* test: add dogecoin device_sub_account test case

* feat: Modify the bitcoin address generation method

* feat: support dogecoin sign and add test case

* chore: modify dogecoin instance id

* test: delete useless test case
@xiaoguang1010 xiaoguang1010 requested a review from XuNeal October 30, 2024 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants