Skip to content

Murisi/fix tx malleability draft2 #1639

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.

Sign up for GitHub

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

Jump to bottom
Closed
murisi wants to merge 182 commits into from
Closed

Murisi/fix tx malleability draft2 #1639

murisi wants to merge 182 commits into from

Conversation

@murisi
Copy link
Collaborator

@murisi murisi commented Jul 2, 2023

Merged #1607 into draft .

batconjurer and others added 30 commits March 29, 2023 09:23
@batconjurer
Drafting
932156c
@batconjurer
Fixed errors in core
3aa2856
@batconjurer
temp
d2d3665
@batconjurer
Some more fixes for amount types in the client code
faedfed
@tzemanovic
core/token: add fn zero
0082f2b
@tzemanovic
core/token: add write_denom
a468d49
@tzemanovic
core/transaction/wrapper: add a todo for later
6bb685d
@tzemanovic
app/shell: use token::Amount::native_whole for min fee conversion
f17e75e
@tzemanovic
core/token: make conv from Amount to u128 falliable
8a64466
@tzemanovic
WIP: shared/pos: update token to TM voting power conv
de17218
@batconjurer
Fixed most of the masp compile issues. May not be logically correct yet
819f319
@batconjurer
[feat]: First compiling refactor. Now for testing
92976fd
@batconjurer
[feat]: Fixed unit tests
b651787
@batconjurer
[chore]: Formatting
2356c5d
@batconjurer
[chore]: Added test coverage for new token types and fixed the bugs t…
b244e1b 
…hey found
@batconjurer
[fix]: Fixed some conversions to amounts in tests. Fixed PoS state ma…
b58d49b 
…chine tests
@batconjurer
[fix]: Fixed the router tests
8efcc32
@batconjurer
[fix]: Cleanup up the wasms
70702a1
@batconjurer
[feat]: Added sub-prefixs to denom storage read/writes
86698a9
@batconjurer
[fix]: Fixed wasm tests
5d10c6c
@batconjurer
[chore] Merging in main
bed746d
@batconjurer
[chore]: Merging in main
8c977cc
@batconjurer
[feat]: Added multitoken support to MASP
a8d9b72
@batconjurer
[fix]: Fixed makefile recipes for abcipp
33b2bf0
@batconjurer
[feat]: Replaced u128 in governance with u256
9d20a96
@batconjurer
[fix]: Incorporating in review suggestions
adaf3a1
@batconjurer
[fix]: Refactored gen_sheilded_transfer to only denominate amounts at…
acc79d2 
… the masp crate boundary
@mariari
Update query_sheilded_balance to handle the multiple denominations
1e3e77d
@mariari
Update query_sheilded_balance to handle the multiple denominations
e71cd44
@batconjurer
Merge branch 'bat/feature/uint256_amount' of github.com:anoma/namada …
88b6ae1 
…into bat/feature/uint256_amount
murisi and others added 29 commits June 23, 2023 20:58
@murisi
Adjusted expected gas usage in proposal test.
7eef159
@murisi
VPs now check that code and data are signed together.
ef34267
@murisi
Make the signature section unmalleable.
1819d16
@batconjurer
[NOTE] Building wasm with wasm-opt ^112 fixes wasm bugs
4531c26
@batconjurer
[fix]: Fixed masp tx deserialization issue
f4c9079
@batconjurer
[fix]: Fixed masp e2e test that tests transfers and queries
72d8e9d
@murisi
Expanded validate_header to check for signature over all sections, an…
a3a8bbc 
…d renamed it to validate_tx.
@Fraccaman
Merge branch 'origin/tomas/fix-optional-prefix-iter' (#1615)
b01c3d3 
* origin/tomas/fix-optional-prefix-iter:
  changelog: add #1615
  ledger/storage/rocksdb: refactor prefix iter code
  ledger/storage/rocksdb: fix iterator without prefix
  test/ledger/storage/rocksdb: add a test for prefix iter
@Fraccaman
Merge branch 'origin/tomas/fix-rollback' (#1616)
489c3cc 
* origin/tomas/fix-rollback:
  ledger/storage/rocksdb: fix rollback to restore deleted keys
  test/ledger/storage/rocksdb: add a test for block rollback
@mariari
Improve consistentcy on pinned tx read
2f398b7 
For some reason Ι believe if we wait an epoch we get pinned
transactions more reliably showing through
@batconjurer
[fix]: Merge of 17.2 complete. Tests passing
e998bce
@batconjurer
[feat]: Merge in main
621b8d1
@batconjurer
Merge branch 'bat/feature/uint256_amount' into draft
cd77fb1
@batconjurer
Merge branch 'origin/bat/feature/uint256-amount' (#1282)
202b1fc
@batconjurer
[fix]: Fixed masp_incentives test startup
5ac86ea
@batconjurer
[fix]: Fixed masp_incentives test startup
55aa4c0
@murisi
Merged in draft.
16b336f
@murisi
Removed the mode flag from the generator script. Corrected the force-…
b588d89 
…alias argument in key generation.
@batconjurer
[fix]: Fixed masp txs and queries e2e test
f32846c
@murisi
Merged in draft.
59a900d
@murisi
Merge branch 'draft' into murisi/move-init-proposal-content-draft
bfd9d41
@murisi
Merge branch 'murisi/move-init-proposal-content-draft' into draft
6ab237a
@murisi
[fix]: Fixed processing of code inside init-proposal transactions.
d232ada
@batconjurer
[fix]: Formatting
340ef76
@batconjurer
Merge branch 'draft' of github.com:anoma/namada into draft
d2b298e
@batconjurer
[fix]: Formatting
3d713fb
@brentstone
evil: fix test_become_validator (not in any branch)
63c6ea6
@murisi
[fix]: Fix failing test-wasm by adding code section and also signing …
d52a137 
…over that.
@murisi
Merge branch 'murisi/fix-tx-malleability' into murisi/fix-tx-malleabi…
a3ab7f1 
…lity-draft2
@murisi murisi closed this Jul 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@murisi @batconjurer @tzemanovic @mariari @brentstone @juped @yito88 @karbyshev @Fraccaman