KeyringController:signTransaction does not cover all check for valid transaction
@mikesposito arbeitet bereits daran.
Seit 11.9.2024.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 413
- Forks
- 308
- Ø Merge
- 1 T. 4 Std.
- Gemergte PRs (30 T.)
- 253
Beschreibung
`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.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Bewertung
Dieses Issue wurde noch nicht bewertet.