-
Notifications
You must be signed in to change notification settings - Fork 4
Add ip tunnel instructions readme #33
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
base: main
Are you sure you want to change the base?
Conversation
aljones15
commented
Feb 2, 2024
- Adds a Table of Contents to the README more inline with a standard README
- Adds a Usage section to the README
- Adds a section in configuration on IP Tunnels
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #33 +/- ##
=======================================
Coverage 92.20% 92.20%
=======================================
Files 2 2
Lines 231 231
=======================================
Hits 213 213
Misses 18 18 Continue to review full report in Codecov by Sentry.
|
README.md
Outdated
|
|
||
| In `config/server.js`: | ||
| ```js | ||
| // accept connections from any IP |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Binding a service to 0.0.0.0 means that your service will listen on all it's network interfaces.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
0.0.0.0 means a lot of different things I've found so thanks for the clarification. Made the change here: f9694da
| In `config/server.js`: | ||
| ```js | ||
| // accept connections from any interface on your device | ||
| config.server.domain = '0.0.0.0'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mattcollier, it seems a little odd to me that this isn't using bindAddr here but domain.