Skip to content

Conversation

@aljones15
Copy link
Contributor

  1. Adds a Table of Contents to the README more inline with a standard README
  2. Adds a Usage section to the README
  3. Adds a section in configuration on IP Tunnels

@aljones15 aljones15 self-assigned this Feb 2, 2024
@codecov-commenter
Copy link

Codecov Report

Merging #33 (adec0ae) into main (447d07c) will not change coverage.
The diff coverage is n/a.

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.

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

README.md Outdated

In `config/server.js`:
```js
// accept connections from any IP
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

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

@aljones15 aljones15 marked this pull request as ready for review February 2, 2024 16:58
In `config/server.js`:
```js
// accept connections from any interface on your device
config.server.domain = '0.0.0.0';
Copy link
Member

@dlongley dlongley Feb 2, 2024

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.

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.

5 participants