Skip to content

rpc/lib: write a test for TLS server#3703

Merged
ebuchman merged 3 commits intodevelopfrom
3700-tls
Jun 22, 2019
Merged

rpc/lib: write a test for TLS server#3703
ebuchman merged 3 commits intodevelopfrom
3700-tls

Conversation

@melekes
Copy link
Contributor

@melekes melekes commented Jun 2, 2019

Refs #3700

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

@melekes melekes requested review from ebuchman and xla as code owners June 2, 2019 02:37
certstrap request-cert -ip "::" --passphrase ""
certstrap sign "::" --CA "tendermint.com" --passphrase ""
mv out/::.crt out/::.key db/remotedb
certstrap request-cert --common-name "remotedb" -ip "127.0.0.1" --passphrase ""
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure --common-name "remotedb" is correct but seems to work

# run the linter
# make lint

# setup certs
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need to regenerate certificates every time

@melekes melekes self-assigned this Jun 5, 2019
@codecov-io
Copy link

Codecov Report

Merging #3703 into develop will decrease coverage by 0.15%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #3703      +/-   ##
===========================================
- Coverage    63.51%   63.35%   -0.16%     
===========================================
  Files          218      218              
  Lines        18239    18239              
===========================================
- Hits         11585    11556      -29     
- Misses        5693     5709      +16     
- Partials       961      974      +13
Impacted Files Coverage Δ
privval/signer_validator_endpoint.go 75.55% <0%> (-12.23%) ⬇️
privval/socket_listeners.go 86.2% <0%> (-10.35%) ⬇️
libs/events/events.go 93.2% <0%> (-4.86%) ⬇️
consensus/replay.go 70.2% <0%> (-2.05%) ⬇️
consensus/reactor.go 71.47% <0%> (-0.71%) ⬇️
consensus/state.go 79.97% <0%> (+0.23%) ⬆️
blockchain/reactor.go 71.49% <0%> (+0.93%) ⬆️

@ebuchman ebuchman merged commit 1b77bf6 into develop Jun 22, 2019
@ebuchman ebuchman deleted the 3700-tls branch June 22, 2019 15:44
@ebuchman
Copy link
Contributor

@melekes this is just a test right, we still need to address #3700 ?

@melekes
Copy link
Contributor Author

melekes commented Jun 22, 2019

@melekes this is just a test right, we still need to address #3700 ?

correct

unclezoro pushed a commit to unclezoro/tendermint that referenced this pull request Sep 6, 2019
* rpc/lib: write a test for TLS server

Refs tendermint#3700

* do not regenerate certificates

* add nolint
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