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

Add debug! trace for the outgoing RPC request #1173

Merged
merged 1 commit into from
Aug 4, 2022

Conversation

romac
Copy link
Member

@romac romac commented Aug 3, 2022

  • Referenced an issue explaining the need for the change
  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Added entry in .changelog/

@romac romac marked this pull request as ready for review August 4, 2022 21:01
@thanethomson thanethomson merged commit 717f680 into v0.23.x Aug 4, 2022
@thanethomson thanethomson deleted the romac/rpc-outgoing-debug branch August 4, 2022 21:38
thanethomson added a commit that referenced this pull request Aug 5, 2022
* Sync README with master

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Sync contributing guidelines with master

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Update README to conform to new Tendermint Core versioning approach

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Import Rust formatting styles from #1120

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* cargo +nightly fmt

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Apply formatting to tools

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Sync GitHub templates with master and remove version-specific fields

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Import cargo audit config from master

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Import ADRs from master

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Pin `time` to <0.3.12 to ensure we stay no-std compatible (#1174)

* Pin `time` to <0.3.12 to ensure we stay no-std compatible

* Apply suggestions from code review

* Update proto/Cargo.toml

Co-authored-by: Thane Thomson <thane@informal.systems>

* Add debug! trace for the outgoing RPC request (#1173)

* Update Prost version to v0.11 (#1171)

* Bump Prost version to v0.11

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Add changelog entry

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Add step in generated-protos-compile job to install protoc

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* shell should be run

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Tell curl to follow redirects

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Fix protoc unzip target

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Install protoc to unprivileged location

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Regenerate protos

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Release v0.23.9 (#1177)

* Bump version to v0.23.9

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Prepare changelog release

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Rebuild changelog

Signed-off-by: Thane Thomson <connect@thanethomson.com>

* Add note on time dependency restriction to changelog

Signed-off-by: Thane Thomson <connect@thanethomson.com>

Co-authored-by: Romain Ruetschi <romain@informal.systems>
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.

2 participants