search

ngoduykhanh / wireguard-ui

Wireguard web interface
MIT License
3.78k stars 464 forks source link

Fix logic error and make some settings optional in wg.conf template. #506

Closed nanhantianyi closed 6 months ago

nanhantianyi commented 6 months ago

Fix logic error and make some settings optional in wg.conf template.

ngoduykhanh commented 6 months ago

Hey @nanhantianyi thanks for the PR. Could you please make your PR description more informative to speed up my PR review process? Which was the issue and why do we need some sitting optional in the config file template? Please also always add an empty line at the end of the file. Thank you!

nanhantianyi commented 6 months ago

Hi @ngoduykhanh ,there is an {{end}} tag in wrong place which make the "Endpoint" config out of the range of ".Client.Enabled". Make “PostUp”、”PreDown“、”PostDown“、”Table“ optional to ensure there is no empty config in generated file if user don't set them.