Add support for format:2 transaction data #12
+16
−3
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Arweave introduced a new format for transaction data but the weavemail app hasn't kept up.
Currently it is unable to view messages stored using the new transaction data format.
This PR adds support for
format:2
transaction data and enables weavemail app to view messages again.The behavior I'm observing is sometimes the weavemail transactions arrive with their
txn.data
field initialized and sometimes they don't. In the cases where.data
is uninitialized the app isn't able to display the message. You'll see an error in the log aboutb64UrlToBuffer()
unable to perform a.replace
on the argument data because it's not a string.This behavior is intermittent, some days the txn's have
.data
initialized sometimes they require.getData()
.