-
- Downloads
Rename GetAllGateways to FetchAllGateways and change return values of FetchAllGateways
After removing `Bitmask.ReloadFirewall()` there are no calls to `b.api.GetAllGateways()` where the calling function actually need the gateways as return values. So I renamed the function to FetchAllGateways. The function now only returns an error (instead of `([]bonafide.Gateway, error)`). This simplifies the code of v5 a bit, as we get rid of yet another function/abstraction. In v3 `gatewayPool.getAll()` now also returns just an error.
parent
4b7392e3
No related branches found
No related tags found
Showing
- pkg/vpn/bonafide/bonafide.go 4 additions, 5 deletionspkg/vpn/bonafide/bonafide.go
- pkg/vpn/bonafide/gateways.go 6 additions, 4 deletionspkg/vpn/bonafide/gateways.go
- pkg/vpn/interface.go 1 addition, 1 deletionpkg/vpn/interface.go
- pkg/vpn/menshen/gateway.go 6 additions, 17 deletionspkg/vpn/menshen/gateway.go
- pkg/vpn/menshen/integration_test.go 18 additions, 12 deletionspkg/vpn/menshen/integration_test.go
- pkg/vpn/openvpn.go 1 addition, 1 deletionpkg/vpn/openvpn.go
Loading
Please register or sign in to comment