Bugfix for issue #41 reported by @reamasesa

Author: d3vilh

controllers/certificates.go

@@ -251,7 +251,9 @@ func (c *CertificatesController) saveClientConfig(keysPath string, name string)
 	cfg.PersistKey = ovClientConfig.PersistKey
 	cfg.RemoteCertTLS = ovClientConfig.RemoteCertTLS
 	cfg.RedirectGateway = ovClientConfig.RedirectGateway
-	// cfg.Proto = ovClientConfig.Proto // this will be set from server config
+	cfg.Proto = ovClientConfig.Proto   // this will be set from client instead of server config
+	cfg.Auth = ovClientConfig.Auth     // this will be set from client instead of server config
+	cfg.Cipher = ovClientConfig.Cipher // this will be set from client instead of server config
 	cfg.Device = ovClientConfig.Device
 	cfg.AuthNoCache = ovClientConfig.AuthNoCache
 	cfg.TlsClient = ovClientConfig.TlsClient
@@ -288,9 +290,9 @@ func (c *CertificatesController) saveClientConfig(keysPath string, name string)
 	serverConfig := models.OVConfig{Profile: "default"}
 	_ = serverConfig.Read("Profile")
 	cfg.Port = serverConfig.Port
-	cfg.Proto = serverConfig.Proto
-	cfg.Auth = serverConfig.Auth
-	cfg.Cipher = serverConfig.Cipher
+	// cfg.Proto = serverConfig.Proto   //Now getting it from client config
+	// cfg.Auth = serverConfig.Auth     //Now getting it from client config
+	// cfg.Cipher = serverConfig.Cipher //Now getting it from client config
 
 	destPath := filepath.Join(state.GlobalCfg.OVConfigPath, "clients", name+".ovpn")
 	if err := SaveToFile(filepath.Join(c.ConfigDir, "openvpn-client-config.tpl"), cfg, destPath); err != nil {

views/ovclient.html

@@ -41,7 +41,7 @@ <h3 class="card-title">Client configuration template for .opvn file</h3>
       </div>
 
       <div class="form-group">
-        <label for="name">Proto</label>
+        <label for="name">Proto</label> <span id="helpBlock" class="text-muted">Don't forget to change server configuration side.</span>
         <input type="text" class="form-control" id="Proto" name="Proto" placeholder="Enter external server protocol"
           value="{{ .Settings.Proto }}">
       </div>
@@ -112,13 +112,13 @@ <h3 class="card-title">Client configuration template for .opvn file</h3>
       </div>
 
       <div class="form-group">
-        <label for="name">Cipher</label>
+        <label for="name">Cipher</label> <span id="helpBlock" class="text-muted">Don't forget to change server configuration side.</span>
         <input type="text" class="form-control" id="Cipher" name="Cipher" placeholder="AES-256-CBC"
           value="{{ .Settings.Cipher }}">
       </div>
 
       <div class="form-group">
-        <label for="name">Auth</label>
+        <label for="name">Auth</label> <span id="helpBlock" class="text-muted">Don't forget to change server configuration side.</span>
         <input type="text" class="form-control" id="Auth" name="Auth" placeholder="SHA512"
           value="{{ .Settings.Auth }}">
       </div>