-
Notifications
You must be signed in to change notification settings - Fork 104
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 state for transactions with a nonce that has already been processed for an address #1038
Comments
wen |
Will this tx be dropped after some time? What would the state be? |
Yes dropped after 256 blocks, unprocessable/unprocessed before that. I see this is ready, when is it set to go live? |
Ready means that we pick it up to do, but at the moment, we are very low in capacity, so it might be some time before we ship it. Having said that we want to prioritize tx status information tasks like this one. |
@andresgalante
The issue I see is with 2. I don't think the API has an endpoint to search for a tx by nonce, which would be required to know whether a tx with the same nonce has been mined already. If we had this endpoint, this feature would then also require an additional network request to assess the tx status, and I don't think the explorer is the place to add and handle this complexity. If we want to assign another tx status for these tx scenarios I think it should be done on the API/Node layer. Perhaps, when an account tx gets mined, we could check all txs for that account and see if there is a tx with the same nonce and in the situation update the state. |
@BLuEScioN Those are fair points. Considering your findings, let's hold this issue for now and work on it when we take on improving tx status |
Bump, any chance this could be prioritized prior to or soon after the Nakamoto activation? I see transactions like these are causing quiet some confusion among new users. |
Do we primarily need this state from the API, then both the explorer and wallets can use it? If so, let's create an issue in the API repo? |
Does this addition mean these transactions can now be filtered out? hirosystems/stacks-blockchain-api#2011 Lets not show pending transactions with a nonce that has already been processed in the explorer or the wallet anymore? If you insist on showing them because they can be used in a reorg... then show them for 6 bitcoin blocks after transmitting or after a transaction with that nonce has been processed. The likelyhood of a reorg after that is close to 0 anyway. |
Please review the existing enhancement issues at the link below before creating a new one to ensure you do not create a duplicate request.
I see #1003 was merged. That is awesome.
I believe no addition was made based on the comment I send about "nonce already used"? I see such a transaction remains pending now.
Is your feature request related to a problem? Please describe.
Here is an a practical example.
Describe the solution you'd like
Although "dropped" isn't the right term here because it could still get mined in case of a reorg I do believe an additional state could be helpful here to indicate the transaction is unlikely to get processed. Without a reorg it will not happen.
"Unprocessable" could be a helpful label? I think it helps with what to expect for a user at least.
Describe alternatives you've considered
The text was updated successfully, but these errors were encountered: