Bug #11286
Endpoint port is mandatory if Endpoint is defined
100%
Description
It's not possible to define endpoint without port, i.e.
EndPoint = 192.168.1.1
Associated revisions
Assume default WG port if empty. Fixes #11286
While here, print a more user-friendly value when peer endpoints are
empty.
Use gettext() on WireGuard endpoint text. Issue #11286
History
Updated by #2
Updated by Jim Pingle 3 months ago
Updated by - Assignee set to Jim Pingle
- Target version set to 2.5.0
In this case we should assume the default port (51820) rather than making the field required. I'll take a look at it.
#3
Updated by Jim Pingle 3 months ago
- Status changed from New to Feedback
- % Done changed from 0 to 100
Applied in changeset df799f2c43441dc80174f6360ecdab0e78b15eb4.
#4
Updated by Jim Pingle 3 months ago
- Status changed from Feedback to In Progress
- % Done changed from 100 to 90
One more little thing, in the tunnel list it isn't assuming the default port in the display. Also it's showing ":" for empty/dynamic peers, should show something more friendly instead.
#5
Updated by Jim Pingle 3 months ago
- Status changed from In Progress to Feedback
- % Done changed from 90 to 100
Applied in changeset e801e55ba199db0cddeb05f5e0b8a0f7ba75c384.
#6
Updated by Jim Pingle 3 months ago
- Status changed from Feedback to Resolved
OK on current snapshot. If the endpoint is filled in and port is blank, the default port is used.
Assume default WG port if empty. Fixes #11286