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

[FEAT] support net mask configuration #270

Closed
1 task done
alonbg opened this issue Jun 15, 2023 · 2 comments
Closed
1 task done

[FEAT] support net mask configuration #270

alonbg opened this issue Jun 15, 2023 · 2 comments

Comments

@alonbg
Copy link

alonbg commented Jun 15, 2023

Is this a new feature request?

  • I have searched the existing issues

Wanted change

few options for supporting this

  1. BYOT (bring your own template) ..
    e.g.
    specify NET_MASK via env
    modify wireguard/templates/server.conf to Address = ${INTERFACE}.1/${NET_MASK:-32}

  2. Support above template and parameter as part of the distribution

  3. extract mask from INTERNAL_SUBNET

Reason for change

required for multiple remote peer support

Proposed code change

for option 3 - a backward compatible suggestion:

modify this line to INTERFACE="$(IFS=/ read _IP _MASK <<<$INTERNAL_SUBNET; echo $_IP/${_MASK:-32})"

@alonbg alonbg added the enhancement New feature or request label Jun 15, 2023
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

@LinuxServer-CI LinuxServer-CI closed this as not planned Won't fix, can't repro, duplicate, stale Sep 27, 2023
@github-actions
Copy link

This issue is locked due to inactivity

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Archived in project
Development

No branches or pull requests

2 participants