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

[BUG] Problem with parsing TX broadcast response #1991

Closed
LNow opened this issue Nov 24, 2021 · 9 comments
Closed

[BUG] Problem with parsing TX broadcast response #1991

LNow opened this issue Nov 24, 2021 · 9 comments
Labels
area:error-handling bug Functionality broken bug-p4 Non-critical functionality broken for few users, or there are clear workarounds

Comments

@LNow
Copy link

LNow commented Nov 24, 2021

Every now and then after submitting transaction instead of proper success information I get an error like to the one below:

Broadcast error: "SyntaxError: Unexpected token b in JSON at position 1 - undefined"
image

Transactions are submitted successfully and they can be found in explorer.
This problem affects STX transfer transactions and contract calls.

Unfortunately I can't reproduce it at will.

Wallet version: v2.20.3
Browser: Firefox Nightly v94.0 (64-bit)

@LNow
Copy link
Author

LNow commented Nov 24, 2021

Another occurrence:
image

@LNow LNow changed the title [BUG] Problem with TX broadcast response parsing. [BUG] Problem with parsing TX broadcast response Nov 24, 2021
@egtalbot
Copy link

Similar error for me on both stackswap and arkadiko swaps
Broadcast error: "SyntaxError: Unexpected token a in JSON at position 2 - undefined"

Wallet version: 2.21.0
Browser: Brave V1.32.106

@aulneau
Copy link
Contributor

aulneau commented Nov 25, 2021

I've also received this error in the desktop wallet

@314159265359879
Copy link
Contributor

314159265359879 commented Nov 25, 2021

I have also seen this error when deploying contracts to mainnet using the explorer sandbox.

@markmhendrickson markmhendrickson added bug Functionality broken bug-p3 Non-critical functionality broken for many users, or there are clear workarounds labels Nov 30, 2021
@obycode
Copy link

obycode commented Jan 11, 2022

I just saw this error today as well. Not sure if these are related or helpful, but checked the console after seeing the error and found:

o204651.ingest.sentry.io/api/5963881/envelope/?sentry_key=XXXXXXXX&sentry_version=7:1 Failed to load resource: the server responded with a status of 429 ()
stacks-node-api.stacks.co/v2/transactions:1 Failed to load resource: the server responded with a status of 524 ()
stacks-node-api.stacks.co/extended/v1/address/XXXXXXXXXXX/balances:1 Failed to load resource: the server responded with a status of 524 ()

@obycode
Copy link

obycode commented Jan 11, 2022

Interestingly, it does appear that the transaction did go through, despite this error.

@egtalbot
Copy link

I just saw this error today as well. Not sure if these are related or helpful, but checked the console after seeing the error and found:

o204651.ingest.sentry.io/api/5963881/envelope/?sentry_key=XXXXXXXX&sentry_version=7:1 Failed to load resource: the server responded with a status of 429 ()
stacks-node-api.stacks.co/v2/transactions:1 Failed to load resource: the server responded with a status of 524 ()
stacks-node-api.stacks.co/extended/v1/address/XXXXXXXXXXX/balances:1 Failed to load resource: the server responded with a status of 524 ()

Without looking at the code, I would think those would be errors from after the transaction is posted

@BowTiedMooneeb
Copy link

I can confirm that these broadcast errors occur frequently when executing CityCoins mining operations over at syvitamining.com. In almost all cases, the transaction actually goes through and shows up in the mempool.

This is a particularly impactful problem when MiamiCoin mining pool participants are submitting STX to the smart contract for mining. We do not have the ability to send STX back from the smart contract, so if someone thinks their transaction failed and they submit it again, they might unintentionally send in twice the amount they intended, which is unfortunately an irrevocable transaction.

@markmhendrickson markmhendrickson added bug-p4 Non-critical functionality broken for few users, or there are clear workarounds and removed bug-p3 Non-critical functionality broken for many users, or there are clear workarounds labels Apr 4, 2022
@markmhendrickson
Copy link
Collaborator

I'm closing and will reopen upon receipt of further reports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:error-handling bug Functionality broken bug-p4 Non-critical functionality broken for few users, or there are clear workarounds
Projects
None yet
Development

No branches or pull requests

8 participants