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

Docs SBAT specify the correct proxy ip address and port #2110

Merged
merged 3 commits into from
Dec 7, 2019

Conversation

kevjue
Copy link
Contributor

@kevjue kevjue commented Dec 6, 2019

Description

Fix to the docs so that it the proxy docker command will use the correct port forwarding rule and instruct to use the external ip address for the proxy.

Tested

Tested using an external validator and verified that it established the necessary number of peers.

Other changes

Related issues

Fixes: #2067

Backwards compatibility

Is backwards compatible.

@codecov
Copy link

codecov bot commented Dec 6, 2019

Codecov Report

Merging #2110 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #2110    +/-   ##
========================================
  Coverage   74.87%   74.87%            
========================================
  Files         281      281            
  Lines        7805     7805            
  Branches      686      972   +286     
========================================
  Hits         5844     5844            
  Misses       1845     1845            
  Partials      116      116
Flag Coverage Δ
#mobile 74.87% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f2db852...cfad93d. Read the comment docs.

@kevjue kevjue changed the title Docs SBAT specify the correct proxy port Docs SBAT specify the correct proxy ip address and port Dec 7, 2019
@kevjue kevjue added automerge Have PR merge automatically when checks pass docs Fixes to any documentation labels Dec 7, 2019
@asaj asaj merged commit 0f22ea6 into master Dec 7, 2019
@asaj asaj deleted the kevjue-fixed_ports branch December 7, 2019 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass docs Fixes to any documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update getting started instructions to use the external IP of the proxy where necessary
2 participants