admin/gluetun
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove duplicate firewall rule for tun0

Browse Source
This commit is contained in:
Quentin McGaw
2020-07-13 02:15:32 +00:00
parent 0fc69e068e
commit 95ad58687d
1 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
internal/firewall/vpn.go
View File

@@ -4,7 +4,6 @@ import (
"context" "context"
"fmt" "fmt"
"github.com/qdm12/private-internet-access-docker/internal/constants"
"github.com/qdm12/private-internet-access-docker/internal/models" "github.com/qdm12/private-internet-access-docker/internal/models"
) )
@@ -32,11 +31,6 @@ func (c *configurator) SetVPNConnections(ctx context.Context, connections []mode
return fmt.Errorf("cannot set VPN connections through firewall: %w", err) return fmt.Errorf("cannot set VPN connections through firewall: %w", err)
} }
// TODO remove elsewhere?
if err := c.acceptOutputThroughInterface(ctx, string(constants.TUN), false); err != nil {
return fmt.Errorf("cannot allow traffic through tunnel: %w", err)
}
c.removeConnections(ctx, connectionsToRemove, defaultInterface) c.removeConnections(ctx, connectionsToRemove, defaultInterface)
if err := c.addConnections(ctx, connectionsToAdd, defaultInterface); err != nil { if err := c.addConnections(ctx, connectionsToAdd, defaultInterface); err != nil {
return fmt.Errorf("cannot set VPN connections through firewall: %w", err) return fmt.Errorf("cannot set VPN connections through firewall: %w", err)