Improvements to wg-quick output for linux.bash.#11
Open
pete4abw wants to merge 4 commits intoWireGuard:masterfrom
Open
Improvements to wg-quick output for linux.bash.#11pete4abw wants to merge 4 commits intoWireGuard:masterfrom
pete4abw wants to merge 4 commits intoWireGuard:masterfrom
Conversation
Replaced use of `<(echo $var)' for `wg setconf -f` and `nft -f` commands. Use of `<(echo $var)' obscured actual input to `wg` and `nft` commands and replaced with /dev/fd/63 which just indicates piped input. After this commit, output will reflect actual commands for `nft` and will echo the config parameters being read by `wg setconf`. Config parameters will also hide Private and Preshared keys like the `wg' command. Before ====== [#] wg setconf wg0 /dev/fd/63 [#] nft -f /dev/fd/63 After ===== [#] wg setconf wg0 /dev/fd/63 wg configuration [Interface] PrivateKey = (hidden) ListenPort = 51820 [Peer] PublicKey = xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx= PresharedKey = (hidden) AllowedIPs = 0.0.0.0/0 Endpoint = xxx.xxx.xxx.xxx:51820 PersistentKeepalive = 25 [#] nft delete table ip wg-quick-wg0
…into wg-quick-linux
zx2c4-bot
force-pushed
the
master
branch
2 times, most recently
from
4574b8e to
0b7d982
Compare
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
1 participant
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.
Replaced use of
<(echo $var)' forwg setconf -fandnft -fcommands. Use of<(echo $var)' obscured actual input towgandnftcommandsand replaced with /dev/fd/63 which just indicates piped input.
After this commit, output will reflect actual commands for
nftand will echo the config parameters being read by
wg setconf.Config parameters will also hide Private and Preshared keys like
the `wg' command.
Before
[#] wg setconf wg0 /dev/fd/63
[#] nft -f /dev/fd/63
After
[#] wg setconf wg0 /dev/fd/63
wg configuration
[Interface]
PrivateKey = (hidden)
ListenPort = 51820
[Peer]
PublicKey = xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx=
PresharedKey = (hidden)
AllowedIPs = 0.0.0.0/0
Endpoint = xxx.xxx.xxx.xxx:51820
PersistentKeepalive = 25
[#] nft delete table ip wg-quick-wg0