Unverified Commit a8a8a811 authored by Jonathan's avatar Jonathan Committed by GitHub
Browse files

Merge pull request #133 from amateo/fix_data_check 

Fix parameters checks in various definitions:
parents ecd6f8d0 5d441f15

manifests/client.pp

+4 −4
Original line number Diff line number Diff line
@@ -4,10 +4,10 @@ define freeradius::client ( 
  Optional[String] $shortname                        = $title,

  Optional[String] $ip                               = undef,

  Optional[String] $ip6                              = undef,

  Enum['*', 'udp', 'tcp'] $proto                     = '*',

  Optional[Enum['*', 'udp', 'tcp']] $proto           = '*',

  Freeradius::Boolean $require_message_authenticator = 'no',

  Optional[String] $virtual_server                   = undef,

  Enum['cisco', 'computone', 'livingston', 'juniper', 'max40xx', 'multitech', 'netserver', 'pathras', 'patton', 'portslave', 'tc', 'usrhiper', 'other'] $nastype = undef,

  Optional[Enum['cisco', 'computone', 'livingston', 'juniper', 'max40xx', 'multitech', 'netserver', 'pathras', 'patton', 'portslave', 'tc', 'usrhiper', 'other']] $nastype = undef,

  Optional[String] $login                            = undef,

  Optional[String] $password                         = undef,

  Optional[String] $coa_server                       = undef,
@@ -20,7 +20,7 @@ define freeradius::client ( 
  Optional[String] $srcip                            = undef,

  Boolean $firewall                                  = false,

  Freeradius::Ensure $ensure                         = present,

  Array $attributes                    = [],

  Variant[Array, Hash, String] $attributes           = [],

  Optional[String] $huntgroups                       = undef,

) {

  $fr_package  = $::freeradius::params::fr_package

manifests/home_server_pool.pp

+1 −1
Original line number Diff line number Diff line 
# Configure home server pools

define freeradius::home_server_pool (

  String $home_server,

  Variant[String, Array[String]] $home_server,

  Enum['fail-over', 'load-balance', 'client-balance', 'client-port-balance', 'keyed-balance'] $type = 'fail-over',

  Optional[String] $virtual_server = undef,

  Optional[String] $fallback       = undef,

manifests/huntgroup.pp

+4 −4
Original line number Diff line number Diff line
@@ -3,7 +3,7 @@ define freeradius::huntgroup ( 
  Freeradius::Ensure $ensure                = present,

  Optional[String] $huntgroup               = $title,

  Optional[Array[String]] $conditions       = [],

  Optional[Integer] $order            = 50,

  Optional[Variant[String, Integer]] $order = 50,

) {

  $fr_basepath = $::freeradius::params::fr_basepath

  $fr_service  = $::freeradius::params::fr_service

manifests/listen.pp

+1 −1
Original line number Diff line number Diff line
@@ -6,7 +6,7 @@ define freeradius::listen ( 
  Optional[String] $ip                                      = undef,

  Optional[String] $ip6                                     = undef,

  Integer $port                                             = 0,

  String $interface                                         = undef,

  Optional[String] $interface                               = undef,

  Array[String] $clients                                    = [],

  Integer $max_connections                                  = 16,

  Integer $lifetime                                         = 0,

manifests/sql.pp

+1 −1
Original line number Diff line number Diff line
@@ -31,7 +31,7 @@ define freeradius::sql ( 
  Optional[Integer] $pool_min                    = 1,

  Optional[Integer] $pool_spare                  = 1,

  Optional[Integer] $pool_idle_timeout           = 60,

  Optional[Float] $pool_connect_timeout          = '3.0',

  Optional[Float] $pool_connect_timeout          = 3.0,

) {

  $fr_package          = $::freeradius::params::fr_package

  $fr_service          = $::freeradius::params::fr_service