Comment 6 for bug 1047362

Revision history for this message
Jazza (y-contact-ubntusso) wrote :

Instead of just whining on a bug tracker I thought I'd take a look at source (trunk). Keep in mind, I don't develop linux software, don't have a linux build environment or version control setup, don't know how to package it, and don't know GTK so with all that in mind:

/src/nm-openvpn-service-defines.h
#define NM_OPENVPN_CONTYPE_OVPN_FILE "ovpn-file"

/src/nm-openvpn-service.c
Add in a new valid property in valid_properties[]
{ NM_OPENVPN_CONTYPE_OVPN_FILE, G_TYPE_STRING, 0, 0, FALSE }

/src/nm-openvpn-service.c under validate_connection_type function
Support NM_OPENVPN_CONTYPE_OVPN_FILE

/src/nm-openvpn-service.c under nm_openvpn_start_openvpn_binary function
Exclude tun default being added to openvpn arguments when specifying OVPN file.
Exclude reneg seconds default being added to openvpn arguments when specifying OVPN file.
Under connection type configuration, check for NM_OPENVPN_CONTYPE_OVPN_FILE and if used, verify specified OVPN file exists and add in the respective OpenVPN argument "--config myovpnfilepath.ovpn" else set error.

According to OpenVPN man page, --config can be mixed with other arguments so I'm guessing it'll still work with management socket, scripts, script security etc?

None of this touches on the UI/GTK stuff though.