William Banfield
c5dc3b267f
test/fuzz: add test to reproduce found fuzz errors ( #6757 )
This change does two things:
1. It fixes the json fuzzer to account for receiving array results. Arrays are returned by the rpc server when the input data is an array.
2. Adds a `fuzz_test.go` file and corresponding `testdata` directory containing the failing test case.
This seems like a reasonable way to add and track previous crash issues in our fuzz test cases. The upcoming stdlib go fuzz tool does effectively this automatically.
3 years ago
dependabot[bot]
93f462ef86
build(deps): Bump codecov/codecov-action from 2.0.1 to 2.0.2 ( #6764 )
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action ) from 2.0.1 to 2.0.2.
- [Release notes](https://github.com/codecov/codecov-action/releases )
- [Changelog](https://github.com/codecov/codecov-action/blob/master/CHANGELOG.md )
- [Commits](https://github.com/codecov/codecov-action/compare/v2.0.1...v2.0.2 )
---
updated-dependencies:
- dependency-name: codecov/codecov-action
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Marko <marbar3778@yahoo.com>
3 years ago
Callum Waters
91e277d7b7
enable pex reactor depending on config param ( #6762 )
3 years ago
Callum Waters
a341a626e0
p2p: avoid blocking on the dequeCh ( #6765 )
3 years ago
William Banfield
c3ae6f5b58
p2p: add coverage for mConnConnection.TrySendMessage ( #6754 )
This change adds additional coverage to the `mConnConnection.TrySendMessage` code path. Adds test to ensure it returns `io.EOF` when closed.
Addresses: #6570
3 years ago
Aleksandr Bezobchuk
a393cf8bab
internal: update blockchain reactor godoc ( #6749 )
3 years ago
Callum Waters
0e2752ae42
light: improve error handling and allow providers to be added ( #6733 )
3 years ago
Callum Waters
97a8f125e0
e2e: allow for both v0 and v1 mempool implementations ( #6752 )
3 years ago
William Banfield
84c15857e4
mempool: return mempool errors to the abci client ( #6740 )
This changes adds an `MempoolError` field to the `ResponseCheckTx`. This will allow clients to understand that their transaction was rejected from the mempool despite passing the ABCI check.
This change also updates the code to make use of early returns to prevent highly nested code blocks. Namely, it returns when the type assertion fails at the beginning of the method, instead of wrapping the entire method in a large if statement. This has a somewhat large effect on the diff as rendered by github.
addresses: #3546
3 years ago
Callum Waters
ed7fa80693
stale bot: ignore issues ( #325 )
3 years ago
JayT106
e70445f942
statesync/event: emit statesync start/end event ( #6700 )
3 years ago
Sam Kleinman
478f5321ad
light: run examples as integration tests ( #6745 )
3 years ago
Marko
08e4e2ed3d
ignore issues for stale bot ( #6747 )
3 years ago
dependabot[bot]
7d63e991c5
build(deps): Bump codecov/codecov-action from 1.5.2 to 2.0.1 ( #6739 )
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action ) from 1.5.2 to 2.0.1.
- [Release notes](https://github.com/codecov/codecov-action/releases )
- [Changelog](https://github.com/codecov/codecov-action/blob/master/CHANGELOG.md )
- [Commits](https://github.com/codecov/codecov-action/compare/v1.5.2...v2.0.1 )
---
updated-dependencies:
- dependency-name: codecov/codecov-action
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Sam Kleinman <garen@tychoish.com>
3 years ago
Sam Kleinman
7638235d33
e2e: longer test harness timeouts ( #6728 )
3 years ago
Callum Waters
2abfe20114
e2e: prevent adding light clients as persistent peers ( #6743 )
3 years ago
William Banfield
0bf7813c4e
fix makefile test target to rely on test makefile ( #6746 )
The Makefile at the root of the repo [includes](cd19ef244e/Makefile (L61))
the Makefile under the `test` package. This fix removes the target defined in the root Makefile in favor of the included one.
3 years ago
Sam Kleinman
ff9038e2ce
e2e: run tests in fewer groups ( #6742 )
3 years ago
Sam Kleinman
00a40835a2
e2e: remove cartesian testing of ipv6 ( #6734 )
Having looked at our network address parsing and connection code, it
really looks like we're not doing anything on top of what the standard
library is doing (both in terms using `net.ParseIP` and also
`net.Dial`,) and I don't think we need to run the tests 2x the number
of times just to run through different areas of the standard
library. I think most of our users are going to be using IPv4, and
would be down to fully remove this dimension as well, if we find it's
making noise, but for now I think it's fine.
3 years ago
JayT106
c4f77ab6d1
fastsync/event: emit fastsync status event when switching consensus/fastsync ( #6619 )
closes #2498
solves part of #3365
Note: difficult to test the event emit in SwitchToFastSync part, might need to change `stateSyncReactor` to an interface in the `nodeImpl` struct
3 years ago
Sam Kleinman
2030875056
e2e: drop single node hybrid configurations ( #6737 )
3 years ago
Sam Kleinman
639e145729
e2e: avoid systematic key-type variation ( #6736 )
3 years ago
Aleksandr Bezobchuk
68ffe8bc64
mempool: add TTL configuration to mempool ( #6715 )
3 years ago
William Banfield
21309ccb7b
clist: add a few basic clist tests ( #6727 )
3 years ago
William Banfield
f70396c6fd
add and run make target for generating existing mocks ( #6732 )
There are many `//go:generate mockery` lines in the source code.
This change adds a make target to invoke these mock generations.
This change also invokes the mock invocations and adds the resulting mocks to the repo.
Related to #5274
3 years ago
William Banfield
fdc246e4a8
libs/clist: revert clear and detach changes while debugging ( #6731 )
3 years ago
Marko
8f9cd23016
update proto files for release ( #318 )
3 years ago
Callum Waters
24f22eeb52
address discrepancies between spec and implementation ( #322 )
3 years ago
Marko
78a0a5fe73
blockchain: error on v2 selection ( #6730 )
## Description
Remove v2 flag from toml
3 years ago
Marko
4f885209aa
RPC: mark grpc as deprecated ( #6725 )
## Description
Mark gRPC as deprecated in the RPC layer.
closes #6718
3 years ago
Callum Waters
6dd0cf92c8
router/statesync: add helpful log messages ( #6724 )
3 years ago
dependabot[bot]
626d9b4fbe
build(deps): Bump actions/stale from 3.0.19 to 4 ( #6726 )
Bumps [actions/stale](https://github.com/actions/stale ) from 3.0.19 to 4.
- [Release notes](https://github.com/actions/stale/releases )
- [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md )
- [Commits](https://github.com/actions/stale/compare/v3.0.19...v4 )
---
updated-dependencies:
- dependency-name: actions/stale
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
3 years ago
dependabot[bot]
f790b6f903
build(deps): bump actions/stale from 3.0.19 to 4 ( #319 )
Bumps [actions/stale](https://github.com/actions/stale ) from 3.0.19 to 4.
- [Release notes](https://github.com/actions/stale/releases )
- [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md )
- [Commits](https://github.com/actions/stale/compare/v3.0.19...v4 )
---
updated-dependencies:
- dependency-name: actions/stale
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
3 years ago
William Banfield
0ff67d6b1e
update spec to reference currently used timestamp type ( #317 )
3 years ago
Sam Kleinman
8addf99f90
e2e: tweak sleep for pertubations ( #6723 )
This tweaks sleeps around pertubations, based on a theory that our
tests with "kill" pertubations restart the nodes fast enough the peers
haven't marked it down when it tries to reconnect. In my local test
runs, this clears out *most* of the test failures that I've seen,
except for one evidence-related test-harness problem (which should be
handled separately.)
3 years ago
Marko
76c6c67734
docs: fix broken links ( #6719 )
## Description
Fix broken links
closes #6695
3 years ago
William Banfield
a46724e4f6
statesync: dispatcher test uses internal channel for timing ( #6713 )
This code change amends the dispatcher tests to read from the dispatcher's `requestCh`. This ensures that a request is waiting when the test calls `dispatcher.respond`.
addresses: #6711
3 years ago
Callum Waters
40fba3960d
add missing context catch and tests ( #6701 )
3 years ago
Callum Waters
36a859ae54
e2e: ensure evidence validator set matches nodes validator set ( #6712 )
3 years ago
Sam Kleinman
ab5c63eff3
statesync: increase dispatcher timeout ( #6714 )
3 years ago
Sam Kleinman
8228936155
e2e: extend timeouts in test harness ( #6694 )
3 years ago
Callum Waters
a12e2bbb60
statesync: use initial height as a floor to backfilling ( #6709 )
3 years ago
dependabot[bot]
11bebfb6a0
build(deps): Bump github.com/google/uuid from 1.2.0 to 1.3.0 ( #6708 )
Bumps [github.com/google/uuid](https://github.com/google/uuid ) from 1.2.0 to 1.3.0.
- [Release notes](https://github.com/google/uuid/releases )
- [Commits](https://github.com/google/uuid/compare/v1.2.0...v1.3.0 )
---
updated-dependencies:
- dependency-name: github.com/google/uuid
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>
3 years ago
William Banfield
4009102e2b
statesync: remove outgoingCalls race condition in dispatcher ( #6699 )
* statesync: remove outgoing calls race condition
3 years ago
William Banfield
cabd916517
Revert "statesync: keep peer despite lightblock query fail ( #6692 )" ( #6696 )
* Revert "statesync: keep peer despite lightblock query fail (#6692 )"
This reverts commit 50b00dff71
.
3 years ago
Marko
363ea56680
abci: remove counter app ( #6684 )
* remove counter app
* remove unneeeded ci
* lint fix
* modify tx sizes
* cleanup docs
* Update abci/cmd/abci-cli/abci-cli.go
Co-authored-by: Callum Waters <cmwaters19@gmail.com>
* Update docs/app-dev/getting-started.md
Co-authored-by: Callum Waters <cmwaters19@gmail.com>
* Update docs/app-dev/getting-started.md
Co-authored-by: Callum Waters <cmwaters19@gmail.com>
* bring back comment
* migrate to kvstore and not persistent
* remove unused func
* test persistent
Co-authored-by: Callum Waters <cmwaters19@gmail.com>
3 years ago
Callum Waters
aa4854ff8f
docs: add docs file for the peer exchange ( #6665 )
3 years ago
dependabot[bot]
aa8f656573
build(deps): bump gaurav-nelson/github-action-markdown-link-check ( #313 )
Bumps [gaurav-nelson/github-action-markdown-link-check](https://github.com/gaurav-nelson/github-action-markdown-link-check ) from 1.0.12 to 1.0.13.
- [Release notes](https://github.com/gaurav-nelson/github-action-markdown-link-check/releases )
- [Commits](https://github.com/gaurav-nelson/github-action-markdown-link-check/compare/1.0.12...1.0.13 )
---
updated-dependencies:
- dependency-name: gaurav-nelson/github-action-markdown-link-check
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
3 years ago
William Banfield
581dd01d47
Update CODEOWNERS to include williambanfield ( #6683 )
3 years ago
William Banfield
50b00dff71
statesync: keep peer despite lightblock query fail ( #6692 )
When a peer responds with no lightblock for the height we queried, we call the [removePeer method](https://github.com/tendermint/tendermint/blob/master/internal/statesync/reactor.go#L339 ). This removes the peer from the [dispatcher's list of called peer's](ad65883152/internal/statesync/dispatcher.go (L159))
. When the dispatcher then receives responses from the removed peer, it [drops their responses](ad65883152/internal/statesync/dispatcher.go (L130))
. These responses may be meaningful or contain a block or data that will help statesync proceed.
[The logs](https://gist.github.com/tychoish/34a1f61eaae3c36c23efc7d0001e805c ), when this change is applied, show an additional 3 networking testnets passing.
addresses: #6691
3 years ago