luci: luci-app-wireguard: Missing option to configure peer address
The Wireguard peer configuration section is missing the address field. I am not aware of configurations omitting the address parameter. It would be handy to set that in the peer configuration so the exported configuration as well as QR code will automatically include the parameter.
At least on Android, the Wireguard app will complain if no address is set and one has to be entered manually after scanning the QR code from the OpenWrt peer config generation page.
In case this is desired behaviour (i.e. to export configs to peers without an address and manually set it on the respective devices), please consider this issue void.
Steps to reproduce:
- Go to: Network → Interfaces → Wireguard → Edit → Peers tab → Edit a peer
Actual behavior:
- The option to set the
addressparameter is not present - As a result, the generated peer config does not contain any
addressparameter
Expected behavior:
- An option to set the
addressparameter for a peer in the peer configuration settings - The
addressparameter is included in the generated peer configs (text and QR code)
Additional Information:
DISTRIB_ID='OpenWrt'
DISTRIB_RELEASE='22.03.1'
DISTRIB_REVISION='r19777-2853b6d652'
DISTRIB_TARGET='rockchip/armv8'
DISTRIB_ARCH='aarch64_generic'
DISTRIB_DESCRIPTION='OpenWrt 22.03.1 r19777-2853b6d652'
DISTRIB_TAINTS=''
About this issue
- Original URL
- State: open
- Created 2 years ago
- Reactions: 3
- Comments: 22 (13 by maintainers)
Commits related to this issue
- Adding an Addresses option to the wireguard peer config generator. Some clients like iOS require this explicitly, and so this change adds the appropriate config with some sensible defaults. Addresses ... — committed to nevumx/luci by nevumx a year ago
- Adding an Addresses option to the wireguard peer config generator. Some clients like iOS require this explicitly, and so this change adds the appropriate config with some sensible defaults. Addresses ... — committed to nevumx/luci by nevumx a year ago
It may be desirable to also make the possibility for setting DNS and MTU
Hi folks, one PR for for the
Addressesfield is here, and there is also one for the DNS setting here. I didn’t do MTU yet, but if someone really wants it, I can add it. First time contributing so let me know if anything is amiss. The PRs will conflict with each other, but I can fix that after as well.The
AddressField is not part ofwgbut ofwg-quicksee also man 8 wg-quick