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

[Bugfix] fix panic at wasm event parse #530

Closed
wants to merge 159 commits into from

Conversation

yun-yeo
Copy link
Contributor

@yun-yeo yun-yeo commented Aug 11, 2021

Summary of changes

replaced #531

Report of required housekeeping

  • Github issue OR spec proposal link
  • Wrote tests
  • Updated API documentation (client/lcd/swagger-ui/swagger.yaml)
  • Added a relevant changelog entry: clog add [section] [stanza] [message]

(FOR ADMIN) Before merging

  • Added appropriate labels to PR
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
  • Confirm added tests are consistent with the intended behavior of changes
  • Ensure all tests pass

alessio and others added 30 commits December 1, 2020 14:26
* upgrade to Cosmos SDK v0.39.2-rc1

* update to cosmos-sdk v0.39.2 RC2

* update to rc3

* update to v0.39.2
* register wasm query failed error code
* fix description for querier
… flag on LCD (#437)

* block querying genesis txs from public node
* update txs query blocking condition for public node
* add tax_caps querier
* add client interface & swagger update

Co-authored-by: Paul Kim <paul@terra.money>
* prevent division by zero, even though no epoch data is stored

* add test case for empty epoch data
* Bump cosmwasm to v0.14.x

* fix lint

* Bump CosmWasm to v0.14.0-beta3

* update swagger-docs

* simulation update

* fix simulation

* fix test binary name

* fix test

* add MigrateMsg parsing & change test contract binary

* fix simulation

* fix msg validation

* fix test

* Update binding interface to reject IBC and Burn message

* implement wasm module migration script
* sperate mint and burn swap pool

* fix lint

* add market migration script
* MsgAuth migration
* Market migration test
* Revert vesting rounding error
* Implement migrate cmd from v0.4 to v0.5
* Fix wrong bank import key to evidence
* bump cosmos-sdk v0.43.0-beta1 & migrate msgauth to cosmos-sdk's authz module

* add address verifier

* revert gitflow

* fix address verifier condition

* remove not used TODO comments

* fix migration script

* fix test

* pass proto codec for wasm simulation
yun-yeo and others added 28 commits August 11, 2021 10:27
* limit wasm gas usage with param

* update comment

* use explicit subCtx
* move oracle SlashWindow check to outside of VotePeriod block

* fix to return BurnBasePool params

* allow to change oracle feeder to other address

* return error when oracle prevote request input is wrong

* remove unnecessary storage access at oracle tally

* sort price ballot at creation time to reduce duplicate sort cost

* fix testcase

* force to spend gas for failed address conversion

* fix to use ValidateFeeder to properly check bonding status of validator at oracle voting

* remove unnecessary oracle price ballot sort

* prevent empty treasury epoch state stored for cumulateive epochs

* prevent to create multiple vesting schedules for a denom

* remove CodeID from the MsgStoreCode

* fix simulation
* bump cosmwasm to v0.15.0

* emit both from_contract and wasm events to support latest & legacy style, but block from_contract to be passed to a contract

* add deprecated comment to from_contract

* invalid replyOn check

* allow to overwrite response data with reply data

* charge gas for wasm events

* simplify gov msg parser

* use wasmvm@v0.15.1

* update wasm event spec

* fix test for wasm gov

* update migration test code to use refect.DeepEqual()

* fix migration test to use json.Marshal for correct comparision

* register UpgradeHandler for v0.5.0

* fix bank encoding

* transfer all luna token to contract for proper simulation

* change require.Equal to assert.JSONEq

clear migration test
* bump sdk version to v0.43.0-rc0

* reuse app creator

* fix estimate_fee request interface unpacker

* remove unused files

* remove unused files

* fix to use new app config at testnet launch

* register all message amino codec to legacy.Cdc because authz module use this to get signbytes

* prevent register codec multiple times

* update cosmos proto for swagger update

* register feegrant amino codec to support legacy service

* change upgrade name to use -rc0
* remove unused sorted check

* rollback mint&burn pool split

* update swagger docs

* update readme
* add contract info query interface

* add testing code for contract info query wasm interface

* remove print
* update changelog

* bump cosmwasm to v0.16.0-rc5 && cosmos-sdk to v0.43.0-rc2

* check reply error

* update test_contract for simulation

* bump cosmwasm to v0.16.0 & cosmos-sdk to v0.43.0-rc3

* add analysis for wasm contract for debugging purpose

* update ibc-go v1.0.0-rc4

* charge contract registering cost

* Update Dockerfile to use CosmWasm 0.16

* add changelog

Co-authored-by: Paul Kim <paul@terra.money>
@yun-yeo yun-yeo closed this Aug 11, 2021
@yun-yeo yun-yeo deleted the bugfix/fix-panic-parse-attribute branch August 11, 2021 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants