From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: network@lists.ipfire.org
Subject: Re: [RFC 2/2] network: add vpn security policies commands
Date: Fri, 14 Jul 2017 13:51:39 -0400
Message-ID: <1500054699.2138.2.camel@ipfire.org>
In-Reply-To: <1500033485.2029.0@mail01.ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0647254309407417546=="
List-Id: <network.lists.ipfire.org>

--===============0647254309407417546==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

On Fri, 2017-07-14 at 13:58 +0200, Jonatan Schlag wrote:
> 
> 
> Am Fr, 14. Jul, 2017 um 1:25 schrieb Michael Tremer <michael.tremer(a)i
> pfire.org>:
> > Hi,
> > 
> > On Thu, 2017-07-13 at 20:33 +0200, Jonatan Schlag wrote:
> >  Signed-off-by: Jonatan Schlag <jonatan.schlag(a)ipfire.org>
> >  ---
> >   src/network | 67
> >  +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--
> >   1 file changed, 65 insertions(+), 2 deletions(-)
> >  
> >  diff --git a/src/network b/src/network
> >  index 154d253..e7b637c 100644
> >  --- a/src/network
> >  +++ b/src/network
> >  @@ -1194,7 +1194,6 @@ cli_reset() {
> >   	fi
> >   
> >   	warning_log "Will reset the whole network
> > configuration!!!"
> >  -
> >   	# Force mode is disabled by default
> >   	local force=0
> >   
> > 
> > This change doesn't have anything to do with the sec pols.
> > 
> >  @@ -1384,13 +1383,77 @@ cli_raw() {
> >   	exit ${EXIT_OK}
> >   }
> >   
> >  +cli_vpn() {
> >  +
> >  +	local action
> >  +	action=${1}
> >  +	shift 1
> > 
> > Just no :)
> 
> This is not understandable :). What just no or why?

It should be:

  local action=${1}
  shift 1

We always do the assignment of the arguments of a function in one line.

> > 
> >  +
> >  +	case "${action}" in
> >  +		security-policies)
> >  +			cli_vpn_security-policies $@
> >  +			;;
> >  +		*)
> >  +			error "Unrecognized argument: ${action}"
> >  +			exit ${EXIT_ERROR}
> >  +			;;
> >  +	esac
> >  +}
> >  +
> >  +cli_vpn_security-policies() {
> > 
> > Try to avoid a dash in the function name.
> > 
> >  +
> >  +	local action
> >  +	local security_policy
> >  +
> >  +	if vpn_security_policy_exists ${1}; then
> >  +
> >  +		security_policy=${1}
> >  +		key=${2}
> >  +		shift 2
> >  +
> >  +		case "${key}" in
> >  +			cipher|compression|integrity|lifetime|pfs
> > |sh
> >  ow)
> >  +				vpn_security_policies_${key}
> >  ${security_policy} $@
> >  +				;;
> >  +			group-type)
> >  +				vpn_security_policies_group_type
> >  ${security_policy} $@
> >  +				;;
> >  +			key-exchange)
> >  +				vpn_security_policies_key_exchang
> > e
> >  ${security_policy} $@
> >  +				;;
> >  +			*)
> >  +				error "Unrecognized argument:
> >  ${key}"
> >  +				exit ${EXIT_ERROR}
> >  +				;;
> >  +		esac
> >  +	else
> >  +		action=${1}
> >  +		shift
> >  +
> >  +		case "${action}" in
> >  +			new)
> >  +				vpn_security_policies_new $@
> >  +				;;
> >  +			destroy)
> >  +				vpn_security_policies_destroy $@
> >  +				;;
> >  +			""|*)
> >  +				if [ -n "${action}" ]; then
> >  +					error "Unrecognized
> >  argument: '${action}'"
> >  +				fi
> >  +				exit ${EXIT_ERROR}
> >  +				;;
> >  +		esac
> >  +	fi
> >  +}
> >  +
> >   # Process the given action
> >   case "${action}" in
> >   	init)
> >   		init_run
> >   		;;
> >   
> >  -	settings|hostname|port|device|zone|start|stop|restart|sta
> > tus
> >  |reset|route)
> >  +	settings|hostname|port|device|zone|start|stop|restart|sta
> > tus
> >  |reset|route|vpn)
> >   		cli_${action} $@
> >   		;;
> >   
> > 
> > -Michael
> 
> Jonatan

--===============0647254309407417546==
Content-Type: application/pgp-signature
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="signature.asc"
MIME-Version: 1.0

LS0tLS1CRUdJTiBQR1AgU0lHTkFUVVJFLS0tLS0KVmVyc2lvbjogR251UEcgdjIKCmlRSWNCQUFC
Q2dBR0JRSlphUVNyQUFvSkVJQjU4UDl2a0FrSEtJY1AvMldkbTJaaGtGdWNtV1pQNFRjQUE0MUYK
a1NvbmFoYm1TOEppUWd2YU03bHdvZHNTS1d1M241RktoTkNkSHU3NEFLSVZCbk1kTVpjNnhCNDJO
ODZjRDR4NApUZWFJeDJkaWlIdzhnWXpLWDlxOVN5NzdrYWlOdnI0QmkvNllaaHZrQnZqS2dlZUlV
NkpYUHZtM2NaZ0lNYWZDCnI1OWdaYlpDWkVlZmFVMEgxd1F0YWRxYUlLaXdLby9wMG5MV1E3czMw
Y1g3dDZjVmV4T09ZZlBkcjc4M25UR3cKS2JEcHhaMXVacUtVZytTQklOUEcwZ3NlQmhGZFNKeGgv
Mnc4d0gyVk9NVzIxQVQvVTdnVk5XQnV1a1NTYm5ZNgo4WmREOVVkSENIUmxpaExRbGNSV1FLY1U1
WWZyNy9GS3A3cldLeFk0Qml4T3MzVnV2M2FYTVNraEJTYU4ySVJBCncvN0txaXhBdVo0NnlRdGRw
V3kzT3BGeGRQR0I4WTN1SEVFWnVoTyt0d2wyRW9DTDZPbnRFcm9QYVYrSmw4c3cKME4vc2tJdURF
WmlMUFRFQWxzQ1YycFQyNFVXNlRTa1ZyMVlmb0tSTFA4OTQxRmRTWGI0TmZlc3gxMDExRVBOMwpH
NkV3NUtDVnYrVU9VZkZmVnZYcUJtRFZLUmdBb1hnbjlNb2xvT3V2aGtwYmlmUTVUUWxVUDA4cmk5
Mm1oQ3B6CmRiZHRLMFZJa3F4SVc4b0huYkJFOXRFdmFJaE1SVDErS04wajI3ZFNTVGtzZ05XdzR6
ZHYzWkVORjVDK3BBL3AKSSsrZEsxVnpxMTkzQk8xMC9xYnA4bU51QUhxVGc3Wm9JUW9MWDZXRzRp
S0dHT1NKaXcyeHRNSjN2U3FkdGgydwpCb0liRU5mZXRpcWZqQTh0YjllVwo9VlpOegotLS0tLUVO
RCBQR1AgU0lHTkFUVVJFLS0tLS0K

--===============0647254309407417546==--