29877384959
25%

Ran 21 Jul 2026 11:34PM UTC

Jobs 1

Files 131

Run time 1min

Badge

Embed ▾

Committed 21 Jul 2026 11:31PM UTC coverage: 24.646% (+0.03%) from 24.612%

Build # 29877384959

Build Type

push

github

Committed by

MagicalTux

Commit Message

rust: swap OKX proxy auths with clientId (Sec-ClientId), not _sign

The app doesn't sign these requests. OKX proxy calls (Crypto/Okx:*) need
only the app's clientId in the Sec-ClientId header (Go withClientID) —
there is no request signature and no per-call credential.

Correcting my earlier _sign/ApiKey approach:
- rest.rs: drop the ApiKey/_sign machinery (struct, sign_query, apply_get,
  apply_post) entirely — the app never used it. Add do_get_params (GET with
  params in the `_` arg + Sec-ClientId), reusing the existing clientId POST
  (do_post) for broadcastTransaction.
- swap.rs: quote/execute_evm/execute_solana/okx_broadcast_swap_tx/
  okx_fetch_order_status now take client_id: Option<&str> and route through
  do_get_params/do_post. No more KeyId/Secret/ApiKey.
- handlers/swap.rs: drop the KeyId/Secret params; source the clientId from
  config_get("walletinfo:clientId") (set via Info:setWalletInfo), same as
  reshare/walletsign.
- Delete tests/rest_auth.rs (tested the removed _sign). Swap tests pass
  client_id.

339 Rust tests + 64/64 Dart, 0 warnings.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

5034 of 20425 relevant lines covered (24.65%)

94.16 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	29877384959.1	21 Jul 2026 11:34PM UTC	131	24.65	GitHub Action Run

KarpelesLab / libwallet / 29877384959
25%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 29877384959

KarpelesLab / libwallet / 29877384959 25%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 29877384959

KarpelesLab / libwallet / 29877384959
25%

README BADGES
x