Write auth file in openvpn loop

cmd/gluetun/main.go

@@ -107,13 +107,6 @@ func _main(background context.Context, args []string) int {
 		fatalOnError(err)
 	}
 
-	err = ovpnConf.WriteAuthFile(
-		allSettings.OpenVPN.User,
-		allSettings.OpenVPN.Password,
-		allSettings.System.UID,
-		allSettings.System.GID)
-	fatalOnError(err)
-
 	defaultInterface, defaultGateway, defaultSubnet, err := routingConf.DefaultRoute()
 	fatalOnError(err)
 
@@ -216,7 +209,7 @@ func _main(background context.Context, args []string) int {
 	unboundDone := make(chan struct{})
 	serverDone := make(chan struct{})
 
-	openvpnLooper := openvpn.NewLooper(ovpnConf, allSettings.OpenVPN, logger, streamMerger, fatalOnError)
+	openvpnLooper := openvpn.NewLooper(ovpnConf, allSettings.OpenVPN, logger, streamMerger, fatalOnError, allSettings.System.UID, allSettings.System.GID)
 	// wait for restartOpenvpn
 	go openvpnLooper.Run(ctx, restartOpenvpn, openvpnDone)
 

internal/openvpn/loop.go

@@ -21,10 +21,12 @@ type looper struct {
 	logger       logging.Logger
 	streamMerger command.StreamMerger
 	fatalOnError func(err error)
+	uid          int
+	gid          int
 }
 
 func NewLooper(conf Configurator, settings settings.OpenVPN, logger logging.Logger,
-	streamMerger command.StreamMerger, fatalOnError func(err error)) Looper {
+	streamMerger command.StreamMerger, fatalOnError func(err error), uid, gid int) Looper {
 	return &looper{
 		conf:         conf,
 		settings:     settings,
@@ -43,6 +45,13 @@ func (l *looper) Run(ctx context.Context, restart <-chan struct{}, done chan<- s
 	}
 	for {
 		openvpnCtx, openvpnCancel := context.WithCancel(ctx)
+		err := l.conf.WriteAuthFile(
+			l.settings.User,
+			l.settings.Password,
+			l.uid,
+			l.gid,
+		)
+		l.fatalOnError(err)
 		stream, waitFn, err := l.conf.Start(openvpnCtx)
 		l.fatalOnError(err)
 		go l.streamMerger.Merge(openvpnCtx, stream,