Add comment and update changelog
Signed-off-by: Kristoffer Dalby <kristoffer@dalby.cc>
This commit is contained in:
parent
356b76fc56
commit
f2da1a1665
2 changed files with 6 additions and 0 deletions
|
@ -7,6 +7,7 @@
|
||||||
- Target Go 1.19 for Headscale [#778](https://github.com/juanfont/headscale/pull/778)
|
- Target Go 1.19 for Headscale [#778](https://github.com/juanfont/headscale/pull/778)
|
||||||
- Target Tailscale v1.30.0 to build Headscale [#780](https://github.com/juanfont/headscale/pull/780)
|
- Target Tailscale v1.30.0 to build Headscale [#780](https://github.com/juanfont/headscale/pull/780)
|
||||||
- Give a warning when running Headscale with reverse proxy improperly configured for WebSockets [#788](https://github.com/juanfont/headscale/pull/788)
|
- Give a warning when running Headscale with reverse proxy improperly configured for WebSockets [#788](https://github.com/juanfont/headscale/pull/788)
|
||||||
|
- Fix subnet routers with Primary Routes [#811](https://github.com/juanfont/headscale/pull/811)
|
||||||
|
|
||||||
## 0.16.4 (2022-08-21)
|
## 0.16.4 (2022-08-21)
|
||||||
|
|
||||||
|
|
|
@ -642,6 +642,11 @@ func (machine Machine) toNode(
|
||||||
|
|
||||||
allowedIPs = append(allowedIPs, machine.EnabledRoutes...)
|
allowedIPs = append(allowedIPs, machine.EnabledRoutes...)
|
||||||
|
|
||||||
|
// TODO(kradalby): This is kind of a hack where we say that
|
||||||
|
// all the announced routes (except exit), is presented as primary
|
||||||
|
// routes. This might be problematic if two nodes expose the same route.
|
||||||
|
// This was added to address an issue where subnet routers stopped working
|
||||||
|
// when we only populated AllowedIPs.
|
||||||
primaryRoutes := []netip.Prefix{}
|
primaryRoutes := []netip.Prefix{}
|
||||||
if len(machine.EnabledRoutes) > 0 {
|
if len(machine.EnabledRoutes) > 0 {
|
||||||
for _, route := range machine.EnabledRoutes {
|
for _, route := range machine.EnabledRoutes {
|
||||||
|
|
Loading…
Reference in a new issue