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

Add LedgerIndex or TransactionId to any emitted exceptions #549

Open
sappenin opened this issue Aug 30, 2024 · 0 comments
Open

Add LedgerIndex or TransactionId to any emitted exceptions #549

sappenin opened this issue Aug 30, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@sappenin
Copy link
Collaborator

Where possible, we should require the LedgerIndex or TransactionId to be included with any particular runtime exception. For example, while trying to deserialize an invalid CryptoCondition in this part of the code, an error here logs a warning, but trying to find the ledger_index that caused this error was difficult.

@sappenin sappenin added the enhancement New feature or request label Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant