William Banfield
0c430ed3a6
wip
2022-01-12 09:22:44 -05:00
William Banfield
f8310ee931
commands: fix rollback test
2022-01-11 18:29:21 -05:00
Sam Kleinman
cc51bf7587
tests: remove in-test logging ( #7558 )
2022-01-11 16:39:31 -05:00
Sam Kleinman
841629f5b7
privval: improve client shutdown to prevent resource leak ( #7544 )
2022-01-11 15:09:19 -05:00
M. J. Fromberger
50ac52e28d
rpc: replace custom context-like argument with context.Context ( #7559 )
...
* Rename rpctypes.Context to CallInfo.
Add methods to attach and recover this value from a context.Context.
* Rework RPC method handlers to accept "real" contexts.
- Replace *rpctypes.Context arguments with context.Context.
- Update usage of RPC context fields to use CallInfo.
2022-01-11 11:47:56 -08:00
M. J. Fromberger
a4d0a43100
rpc: refactor the HTTP POST handler ( #7555 )
...
No functional changes.
- Pull out a some helper code to simplify the control flow within the body of
the HTTP request handler.
- Front-load the URL path check so it does not get repeated for each request.
2022-01-11 11:04:55 -08:00
Sam Kleinman
5bf1bdcfb4
reactors: skip log on some routine cancels ( #7556 )
2022-01-11 12:56:52 -05:00
M. J. Fromberger
7f8b75e1ee
rpc: replace anonymous arguments with structured types ( #7552 )
...
Instead of using anonymous maps, define tagged struct types for JSON argument
encoding. This allows us to have the encoding rules we want without tmjson.
This commit handles the "easy" cases. BroadcastEvidence is omitted here,
because it depends on the interface encoding rules from tmjson. I will address
that in a forthcoming change.
2022-01-11 06:37:38 -08:00
Kene
2f858f1448
node: new concrete type for seed node implementation ( #7521 )
...
Defines a different concrete type that satisfies the service interface for a seed node.
update the seed node unit test to assert the new type.
Fixes #6775
2022-01-11 14:07:49 +01:00
M. J. Fromberger
6291d22f46
rpc: simplify the JSON-RPC client Caller interface ( #7549 )
...
* Update Caller interface and its documentation.
* Rework MapToRequest as ParamsToRequest.
The old interface returned the result as well as populating it. Nothing was
using this, so drop the duplicated value from the return signature. Clarify the
documentation on the Caller type.
Rework the MapToRequest helper to take an arbitrary value instead of only a
map. This is groundwork for getting rid of the custom marshaling code. For now,
however, the implementation preserves the existing behaviour for the map, until
we can replace those.
2022-01-10 13:56:43 -08:00
Sam Kleinman
692701a551
abci: socket server shutdown response handler ( #7547 )
2022-01-10 16:26:40 -05:00
Sam Kleinman
d331a08607
statesync: use specific testing.T logger for tests ( #7543 )
2022-01-10 15:38:20 -05:00
M. J. Fromberger
8e58c564c0
rpc: collapse Caller and HTTPClient interfaces. ( #7548 )
...
These two interfaces are identical, and besides HTTPClient being confusingly
named, all but one location uses Caller. Update that one location, and drop the
redundant interface.
2022-01-10 11:52:52 -08:00
M. J. Fromberger
211d755aca
rpc: remove positional parameter encoding from clients ( #7545 )
...
Apart from the tests for the websocket client, positional parameters are not
used by RPC clients. The server supports both arrays and objects, but the
client only needs to provide one or the other.
2022-01-10 11:20:30 -08:00
Sam Kleinman
0f3f2aa4bc
log: remove support for traces ( #7542 )
2022-01-10 13:56:42 -05:00
Sam Kleinman
f19f84bc8c
test: uniquify prom IDs ( #7540 )
2022-01-10 13:03:53 -05:00
dependabot[bot]
6c669b70a4
build(deps): Bump technote-space/get-diff-action from 5 to 6.0.1 ( #7535 )
...
Bumps [technote-space/get-diff-action](https://github.com/technote-space/get-diff-action ) from 5 to 6.0.1.
- [Release notes](https://github.com/technote-space/get-diff-action/releases )
- [Changelog](https://github.com/technote-space/get-diff-action/blob/main/.releasegarc )
- [Commits](https://github.com/technote-space/get-diff-action/compare/v5...v6.0.1 )
---
updated-dependencies:
- dependency-name: technote-space/get-diff-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>
2022-01-10 11:46:41 +01:00
M. J. Fromberger
366ab1947a
Replace uses of libs/json with encoding/json. ( #7534 )
...
Where possible, replace uses of the custom JSON library with the standard
library. The custom library treats interface and unnamed lteral types
differently, so this change avoids those even where it would probably be safe
to switch them.
2022-01-08 08:47:26 -08:00
Sam Kleinman
d5c39f907d
test/factory: pass testing.T around rather than errors for test fixtures ( #7518 )
2022-01-07 15:51:39 -05:00
Sam Kleinman
90cf742065
pex: regularize reactor constructor ( #7532 )
2022-01-07 13:52:11 -05:00
Sam Kleinman
aa76a367e0
blocksync: standardize construction process ( #7531 )
2022-01-07 13:40:08 -05:00
Sam Kleinman
ae7a76a175
evidence: reactor constructor ( #7533 )
2022-01-07 13:28:23 -05:00
Sam Kleinman
059b38afe4
mempool: refactor mempool constructor ( #7530 )
2022-01-07 12:49:22 -05:00
Sam Kleinman
fc36c7782f
statesync: reactor and channel construction ( #7529 )
2022-01-07 12:04:17 -05:00
Sam Kleinman
10402b728f
consensus+p2p: change how consensus reactor is constructed ( #7525 )
2022-01-07 10:59:10 -05:00
Sam Kleinman
74a8941854
consensus: remove reactor options ( #7526 )
2022-01-07 09:15:35 -05:00
Sam Kleinman
569629486b
tests: remove panics from test fixtures ( #7522 )
2022-01-06 17:34:32 -05:00
Sam Kleinman
a91d3cb894
e2e: make tx test more stable ( #7523 )
2022-01-06 15:43:30 -05:00
Sam Kleinman
0ae974e639
libs/os: remove trap signal ( #7515 )
2022-01-06 08:26:37 -05:00
Sam Kleinman
386c3a0ff7
e2e: constrain test parallelism and reporting ( #7516 )
2022-01-05 17:45:16 -05:00
Sam Kleinman
752a3a6c24
types: tests should not panic ( #7506 )
2022-01-05 14:21:40 -05:00
Sam Kleinman
69f0a3b1c0
e2e: avoid global test context ( #7512 )
2022-01-05 13:35:27 -05:00
Sam Kleinman
332163ede6
testing: remove background contexts ( #7509 )
2022-01-05 12:42:57 -05:00
Sam Kleinman
8564c5079f
e2e: use more simple strings for generated transactions ( #7513 )
2022-01-05 12:17:23 -05:00
Sam Kleinman
4137c16d3f
privval: improve test hygine ( #7511 )
2022-01-05 11:58:14 -05:00
Sam Kleinman
f2cc496f09
testing: pass testing.T to assert and require always, assertion cleanup ( #7508 )
2022-01-05 09:25:08 -05:00
Sam Kleinman
3c8955e4b8
errors: formating cleanup ( #7507 )
2022-01-04 16:11:28 -05:00
Sam Kleinman
a67e0e6c43
light: remove global context from tests ( #7505 )
2022-01-04 13:35:49 -05:00
Sam Kleinman
5c0abb5367
testing: use scoped logger for all public packages ( #7504 )
2022-01-04 12:56:17 -05:00
Sam Kleinman
430817d9e9
types: remove panic from block methods ( #7501 )
2022-01-03 16:14:26 -05:00
M. J. Fromberger
5f85c8f536
types: fix path handling in node key tests ( #7493 )
...
These tests use a deterministic and unseeded random source to generate
non-colliding filenames for testing. When testing locally, this means tests are
not hermetic from one run to the next.
Use proper temp directories, and clean up after they're done.
2022-01-03 09:02:14 -08:00
M. J. Fromberger
7cdf560173
config: add a Deprecation annotation to P2PConfig.Seeds. ( #7496 )
2021-12-27 16:50:47 -08:00
dependabot[bot]
41bfcfeb31
build(deps): Bump docker/login-action from 1.10.0 to 1.12.0 ( #7494 )
...
Bumps [docker/login-action](https://github.com/docker/login-action ) from 1.10.0 to 1.12.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a href="https://github.com/docker/login-action/releases ">docker/login-action's releases</a>.</em></p>
<blockquote>
<h2>v1.12.0</h2>
<ul>
<li>ECR: only set credentials if username and password are specified (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/128 ">#128</a>)</li>
<li>Refactor to use aws-sdk v3 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/128 ">#128</a>)</li>
</ul>
<h2>v1.11.0</h2>
<ul>
<li>ECR: switch implementation to use the AWS SDK (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/126 ">#126</a>)</li>
<li><code>ecr</code> input to specify whether the given registry is ECR (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/123 ">#123</a>)</li>
<li>Test against Windows runner (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/126 ">#126</a>)</li>
<li>Update instructions for Google registry (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/127 ">#127</a>)</li>
<li>Update dev workflow (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/111 ">#111</a>)</li>
<li>Small changes for GHCR doc (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/86 ">#86</a>)</li>
<li>Update dev dependencies (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/85 ">#85</a>)</li>
<li>Bump ansi-regex from 5.0.0 to 5.0.1 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/101 ">#101</a>)</li>
<li>Bump tmpl from 1.0.4 to 1.0.5 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/100 ">#100</a>)</li>
<li>Bump <code>@actions/core</code> from 1.4.0 to 1.6.0 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/94 ">#94</a> <a href="https://github-redirect.dependabot.com/docker/login-action/issues/103 ">#103</a>)</li>
<li>Bump codecov/codecov-action from 1 to 2 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/88 ">#88</a>)</li>
<li>Bump hosted-git-info from 2.8.8 to 2.8.9 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/83 ">#83</a>)</li>
<li>Bump node-notifier from 8.0.0 to 8.0.2 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/82 ">#82</a>)</li>
<li>Bump ws from 7.3.1 to 7.5.0 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/81 ">#81</a>)</li>
<li>Bump lodash from 4.17.20 to 4.17.21 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/80 ">#80</a>)</li>
<li>Bump y18n from 4.0.0 to 4.0.3 (<a href="https://github-redirect.dependabot.com/docker/login-action/issues/79 ">#79</a>)</li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="42d299facehttps://github-redirect.dependabot.com/docker/login-action/issues/130 ">#130</a> from crazy-max/ci-workflow</li>
<li><a href="4858b0b5ea1d7d8649e7https://github-redirect.dependabot.com/docker/login-action/issues/128 ">#128</a> from Flydiverny/aws-sdk-v3</li>
<li><a href="58855695bbd9927c4142https://github-redirect.dependabot.com/docker/login-action/issues/123 ">#123</a> from crazy-max/ecr-input</li>
<li><a href="b9a4d91ee5b20b9f5e31https://github-redirect.dependabot.com/docker/login-action/issues/126 ">#126</a> from crazy-max/aws-sdk</li>
<li><a href="cb21399f71faae4d66654d84a3c20fhttps://github-redirect.dependabot.com/docker/login-action/issues/127 ">#127</a> from crazy-max/carry-124</li>
<li>Additional commits viewable in <a href="https://github.com/docker/login-action/compare/v1.10.0...v1.12.0 ">compare view</a></li>
</ul>
</details>
<br />
[](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores )
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
</details>
2021-12-27 17:42:10 +00:00
dependabot[bot]
a86965679d
build(deps): Bump github.com/rs/cors from 1.8.0 to 1.8.2 ( #7484 )
...
Bumps [github.com/rs/cors](https://github.com/rs/cors ) from 1.8.0 to 1.8.2.
- [Release notes](https://github.com/rs/cors/releases )
- [Commits](https://github.com/rs/cors/compare/v1.8.0...v1.8.2 )
---
updated-dependencies:
- dependency-name: github.com/rs/cors
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: M. J. Fromberger <fromberger@interchain.io >
2021-12-22 09:00:50 -08:00
Carlos Rodriguez
2f5ad5f8cc
docs: update go ws code snippets ( #7486 )
...
* doc: fix typos in /tx_search and /tx.
* docs: update of go snippets for subscribe and unsubscribe operations
Co-authored-by: Carlos Rodriguez <crodveg@gmail.com >
2021-12-22 08:13:53 -08:00
sgmoore
e7afc57a6a
Remove extra white space ( #7477 )
2021-12-19 08:07:27 -08:00
Sam Kleinman
1630d1cf3e
privval: remove panics in privval implementation ( #7475 )
2021-12-17 16:28:32 -05:00
Sam Kleinman
3b6c3890d6
rfc: don't panic ( #7472 )
2021-12-17 16:15:01 -05:00
M. J. Fromberger
cea8bcc9bb
Remove the "URI" RPC client. ( #7474 )
...
The JSON-RPC endpoint accepts requests via URL (GET) and JSON (POST). There is
no real point in having client libraries for both modes.
A search of the SDK and on GitHub suggests that most usage is via the JSON
client (via the New constructor) or websocket (NewWS), and the only uses I
found of the NewURI client constructor are in copies of our own test code.
This does not change the functionalitiy of the server, so curl and other
URL-based clients in other languages will still function as before.
2021-12-17 09:56:41 -08:00
Sam Kleinman
9a0dbdbf13
libs/rand: remove custom seed function ( #7473 )
2021-12-17 12:15:27 -05:00
