MetaMask / MetaMask/core

KeyringController:signTransaction does not cover all check for valid transaction

Aperta
#800 0 commenti 0 reazioni 1 assegnatario Vedi su GitHub

@mikesposito ci sta già lavorando.

Dal 11/9/2024.

team-wallet-framework wf-bugs
Lingua principale
TypeScript
Stelle
413
Fork
308
Merge medio
1g 4h
PR unite (30g)
253

Descrizione

`KeyringController:signTransaction` fails in scenario like

```
const signedTx = await keyringController.signTransaction({}, account);
```

But not in scenario like:

```
const txParams = {
// no txParams passed
};
const unsignedEthTx = TransactionFactory.fromTxData(txParams, {
common: new Common(commonConfig),
freeze: false,
});
expect(unsignedEthTx.v).toBeUndefined();
const signedTx = await keyringController.signTransaction(
unsignedEthTx,
account,
);
```

Also the error message passed for wrong transaction is not very useful.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.