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

Limit prepare execute gas #107

Merged
merged 4 commits into from
Jun 14, 2021
Merged

Limit prepare execute gas #107

merged 4 commits into from
Jun 14, 2021

Conversation

taobun
Copy link
Member

@taobun taobun commented Jun 14, 2021

  • Add maxOwasmGas for validate basic on MsgRequestData, not you gas too much
  • Explicit response packet timeout.

@taobun taobun merged commit 6a2c43b into master Jun 14, 2021
@taobun taobun deleted the limit-prepare-execute-gas branch June 14, 2021 12:00
vuong177 pushed a commit to vuong177/chain that referenced this pull request Feb 5, 2022
…ocol#107)

* make changes and start fixing tests

* add msg server test

* change to revert on SendCoins fail

* update interfaces and fix tests

* self review fixes

* Update modules/core/04-channel/types/acknowledgement.go

* Add Changelog

* update docs

* Update CHANGELOG.md

Co-authored-by: Aditya <adityasripal@gmail.com>

* add note for async acks

Co-authored-by: Aditya Sripal <adityasripal@gmail.com>
RogerKSI pushed a commit that referenced this pull request Mar 18, 2024
Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.8.4 to 1.9.0.
- [Release notes](https://github.com/stretchr/testify/releases)
- [Commits](stretchr/testify@v1.8.4...v1.9.0)

---
updated-dependencies:
- dependency-name: github.com/stretchr/testify
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

4 participants