Add Amazon Linux support - #534
Open
CW-B-W wants to merge 9 commits into
Open
Conversation
angristan
reviewed
Jan 8, 2025
|
|
||
| # Enable WireGuard kernel module | ||
| echo "Enabling WireGuard kernel module..." | ||
| modprobe wireguard |
Author
There was a problem hiding this comment.
It's unnecessary.
I have removed this section and committed.
Thanks!
Owner
|
Thanks, can you update the readme as well? |
Author
Sure, updated. |
Resolved merge conflict in wireguard-install.sh by keeping upstream's installPackages function usage for better error handling.
…n wireguard-manager.sh - Remove duplicate SERVER_PORT question that appeared twice - Add default SERVER_PORT=51820 before port prompt (consistent with wireguard-install.sh) - Update getHomeDirForClient to accept CLIENT_NAME parameter (fixes function signature mismatch)
The ec2 commit dd5a6f5 added curl to all package installation commands since the script uses curl on line 119 to get public IP. The upstream version removed curl, causing the script to fail if curl isn't installed. This preserves the ec2 modification while keeping upstream's installPackages function for better error handling.
…ewing - Remove SERVER_PORT=51820 default, allow user to enter desired port - Add IP and port conflict checks for existing WireGuard interfaces - Add 'Show user configuration' option with normal/VLAN modes - Normal mode: Shows full VPN configuration with DNS - VLAN mode: Shows subnet-only config without DNS, adds PersistentKeepalive - Add PersistentKeepalive = 25 to all client configurations - Update both wireguard-install.sh and wireguard-manager.sh scripts
- Detect default NIC MTU and suggest WireGuard MTU as (NIC_MTU - 80) - Allow user to configure MTU during interface creation - Store WG_MTU in parameters for consistency - Apply configured MTU to both server and client configurations - Remove hardcoded MTU=1420 in favor of calculated defaults
…gement to wireguard-manager.sh
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tested on
Amazon Linux 2023&Amazon Linux 2