Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

Implement support for read-only contract queries in nodeos #10189

Merged
jgiszczak merged 30 commits into from
May 25, 2021
Merged

Implement support for read-only contract queries in nodeos #10189

jgiszczak merged 30 commits into from
May 25, 2021

Conversation

@jgiszczak
Copy link
Contributor

@jgiszczak jgiszczak commented Mar 26, 2021

Change Description

Implement support for read-only contract queries in nodeos.

Change Type

Select ONE:

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • Existing Tests
  • Test Framework
  • CI System
  • Other

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

heifner
heifner reviewed Mar 27, 2021
View reviewed changes
plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
heifner
heifner reviewed Mar 27, 2021
View reviewed changes
plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
nickjjzhao and others added 16 commits March 31, 2021 20:12
@nickjjzhao
Add resulting URL for read-only queries into cleos
1942d92
@nickjjzhao
Add a transaction in cleos for contract query
ac252a7
@nickjjzhao
Print contract query output as JSON
3ea0af1
@nickjjzhao
Make new cleos command contract_query as a subcommand of get
1fdde7b
@nickjjzhao
Merge pull request #10221 from EOSIO/read_only_query_EPE-747
008b2a3 
Implement cleos support for new RPC API
@jgiszczak
Follow through on new read_only flag in producer plugin
08f3d71
@jgiszczak
Merge branch 'read_only_query_EPE-746' of github.com:/EOSIO/eos into …
a18ec6c 
…read_only_query_EPE-746
@nickjjzhao
Fix a bug that fails to invoke new RPC API get_contract_query
a9b6faf
@nickjjzhao
Merge pull request #10223 from EOSIO/read_only_query_EPE-747
1623b47 
Update cleos support for new RPC API
@nickjjzhao
Use updated structure of get_contract_query_params in cleos
1aa6b10
@nickjjzhao
Merge pull request #10225 from EOSIO/read_only_query_EPE-747
7cf0077 
Use updated structure of get_contract_query_params in cleos
@jgiszczak
Use read_only flag in producer plugin initialize
aafaec5
@jgiszczak
Add read-only query integration test
9bd1d7f
@jgiszczak
Merge develop
836a641
@jgiszczak
Add read-only query integration test contracts
2054d61
@jgiszczak
Merge branch 'develop' into read_only_query_EPE-746
00cb72e
@jgiszczak jgiszczak self-assigned this May 13, 2021
@jgiszczak jgiszczak marked this pull request as ready for review May 13, 2021 14:24
heifner
heifner suggested changes May 14, 2021
View reviewed changes
plugins/chain_interface/include/eosio/chain/plugin_interface.hpp Outdated Show resolved Hide resolved
plugins/chain_plugin/chain_plugin.cpp Outdated Show resolved Hide resolved
plugins/chain_plugin/chain_plugin.cpp Outdated Show resolved Hide resolved
plugins/chain_api_plugin/chain_api_plugin.cpp Outdated Show resolved Hide resolved
plugins/chain_plugin/chain_plugin.cpp Show resolved Hide resolved
plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
plugins/producer_plugin/producer_plugin.cpp Show resolved Hide resolved
plugins/producer_plugin/producer_plugin.cpp Show resolved Hide resolved
unittests/contracts/CMakeLists.txt Show resolved Hide resolved
@jgiszczak jgiszczak requested a review from dimas1185 May 17, 2021 15:11
@brianjohnson5972 brianjohnson5972 requested a review from heifner May 20, 2021 21:21
victorj8
victorj8 reviewed May 20, 2021
View reviewed changes
unittests/contracts/read_only_query_tests.abi Show resolved Hide resolved
heifner
heifner reviewed May 24, 2021
View reviewed changes
plugins/chain_plugin/include/eosio/chain_plugin/chain_plugin.hpp Outdated Show resolved Hide resolved
jgiszczak added 2 commits May 24, 2021 11:50
@jgiszczak
Don't return failure traces by default.
df09841
@jgiszczak
Update integration test to push transaction API.
822d90f 
Test verifies new cleos option --return-failure-trace is accepted but
does not test its functionality.
@jgiszczak jgiszczak merged commit f858206 into develop May 25, 2021
@jgiszczak jgiszczak deleted the read_only_query_EPE-746 branch May 25, 2021 06:00
@softprofe softprofe mentioned this pull request Jan 7, 2022
Merged
12 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@brianjohnson5972 brianjohnson5972 Awaiting requested review from brianjohnson5972

@dimas1185 dimas1185 Awaiting requested review from dimas1185

3 more reviewers

@victorj8 victorj8 victorj8 left review comments

@heifner heifner heifner approved these changes

@larryk85 larryk85 larryk85 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jgiszczak jgiszczak

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@jgiszczak @victorj8 @heifner @larryk85 @nickjjzhao
Morty Proxy This is a proxified and sanitized view of the page, visit original site.