KeyringController:signTransaction does not cover all check for valid transaction
@mikesposito ci sta già lavorando.
Dal 11/9/2024.
- 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
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Valutazione
Questa issue non è ancora stata valutata.