Skip to content

Commit d722f9a

Browse files
bastelfreakbastelfreak
committed
port: Enforce Stdlib::Port datatype
1 parent 56bc346 commit d722f9a

18 files changed

+30
-30
lines changed

‎REFERENCE.md

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1031,7 +1031,7 @@ Default value: `$postgresql::params::listen_addresses`
10311031

10321032
##### <a name="-postgresql--server--port"></a>`port`
10331033

1034-
Data type: `Variant[String[1], Stdlib::Port]`
1034+
Data type: `Stdlib::Port`
10351035

10361036
Specifies the port for the PostgreSQL server to listen on.
10371037
Note: The same port number is used for all IP addresses the server listens on.
@@ -1946,7 +1946,7 @@ Default value: `$postgresql::server::psql_path`
19461946

19471947
##### <a name="-postgresql--server--default_privileges--port"></a>`port`
19481948

1949-
Data type: `Variant[String[1], Stdlib::Port]`
1949+
Data type: `Stdlib::Port`
19501950

19511951
Specifies the port to access the server. Default value: The default user for the module, usually '5432'.
19521952

@@ -2051,7 +2051,7 @@ Default value: `undef`
20512051

20522052
##### <a name="-postgresql--server--extension--port"></a>`port`
20532053

2054-
Data type: `Optional[Variant[String[1], Stdlib::Port]]`
2054+
Data type: `Optional[Stdlib::Port]`
20552055

20562056
Port to use when connecting.
20572057

@@ -2178,7 +2178,7 @@ Default value: `$postgresql::server::user`
21782178

21792179
##### <a name="-postgresql--server--grant--port"></a>`port`
21802180

2181-
Data type: `Variant[String[1], Stdlib::Port]`
2181+
Data type: `Stdlib::Port`
21822182

21832183
Port to use when connecting.
21842184

@@ -2280,7 +2280,7 @@ Default value: `$postgresql::server::user`
22802280

22812281
##### <a name="-postgresql--server--grant_role--port"></a>`port`
22822282

2283-
Data type: `Variant[String[1], Stdlib::Port]`
2283+
Data type: `Stdlib::Port`
22842284

22852285
Port to use when connecting.
22862286

@@ -2361,7 +2361,7 @@ Default value: `$postgresql::server::listen_addresses`
23612361

23622362
##### <a name="-postgresql--server--instance--config--port"></a>`port`
23632363

2364-
Data type: `Variant[String[1], Stdlib::Port]`
2364+
Data type: `Stdlib::Port`
23652365

23662366
Specifies the port for the PostgreSQL server to listen on.
23672367
Note: The same port number is used for all IP addresses the server listens on. Also, for Red Hat systems and early Debian systems,
@@ -2785,7 +2785,7 @@ Default value: `$postgresql::server::psql_path`
27852785

27862786
##### <a name="-postgresql--server--instance--late_initdb--port"></a>`port`
27872787

2788-
Data type: `Variant[String[1], Stdlib::Port]`
2788+
Data type: `Stdlib::Port`
27892789

27902790
Specifies the port for the PostgreSQL server to listen on.
27912791
Note: The same port number is used for all IP addresses the server listens on. Also, for Red Hat systems and early Debian systems,
@@ -2844,7 +2844,7 @@ Default value: `$postgresql::server::psql_path`
28442844

28452845
##### <a name="-postgresql--server--instance--passwd--port"></a>`port`
28462846

2847-
Data type: `Variant[String[1], Stdlib::Port]`
2847+
Data type: `Stdlib::Port`
28482848

28492849
Specifies the port for the PostgreSQL server to listen on.
28502850
Note: The same port number is used for all IP addresses the server listens on. Also, for Red Hat systems and early Debian systems,
@@ -2982,7 +2982,7 @@ Default value: `$postgresql::server::user`
29822982

29832983
##### <a name="-postgresql--server--instance--service--port"></a>`port`
29842984

2985-
Data type: `Variant[String[1], Stdlib::Port]`
2985+
Data type: `Stdlib::Port`
29862986

29872987
Specifies the port for the PostgreSQL server to listen on.
29882988
Note: The same port number is used for all IP addresses the server listens on. Also, for Red Hat systems and early Debian systems,
@@ -3219,7 +3219,7 @@ Default value: `$postgresql::server::user`
32193219

32203220
##### <a name="-postgresql--server--reassign_owned_by--port"></a>`port`
32213221

3222-
Data type: `Variant[String[1], Stdlib::Port]`
3222+
Data type: `Stdlib::Port`
32233223

32243224
Port to use when connecting.
32253225

@@ -3465,7 +3465,7 @@ Default value: `$postgresql::server::default_database`
34653465

34663466
##### <a name="-postgresql--server--role--port"></a>`port`
34673467

3468-
Data type: `Optional[Variant[String[1], Stdlib::Port]]`
3468+
Data type: `Optional[Stdlib::Port]`
34693469

34703470
Port to use when connecting.
34713471

@@ -3700,7 +3700,7 @@ Default value: `undef`
37003700

37013701
##### <a name="-postgresql--server--table_grant--port"></a>`port`
37023702

3703-
Data type: `Optional[Variant[String[1], Stdlib::Port]]`
3703+
Data type: `Optional[Stdlib::Port]`
37043704

37053705
Port to use when connecting.
37063706

‎manifests/server.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@
135135
String[1] $default_database = $postgresql::params::default_database,
136136
Hash $default_connect_settings = $postgresql::globals::default_connect_settings,
137137
Optional[Variant[String[1], Array[String[1]]]] $listen_addresses = $postgresql::params::listen_addresses,
138-
Variant[String[1], Stdlib::Port] $port = $postgresql::params::port,
138+
Stdlib::Port $port = $postgresql::params::port,
139139
String[1] $ip_mask_deny_postgres_user = $postgresql::params::ip_mask_deny_postgres_user,
140140
String[1] $ip_mask_allow_all_users = $postgresql::params::ip_mask_allow_all_users,
141141
Array[String[1]] $ipv4acls = $postgresql::params::ipv4acls,

‎manifests/server/default_privileges.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
String $schema = 'public',
2929
String $psql_db = $postgresql::server::default_database,
3030
String $psql_user = $postgresql::server::user,
31-
Variant[String[1], Stdlib::Port] $port = $postgresql::server::port,
31+
Stdlib::Port $port = $postgresql::server::port,
3232
Hash $connect_settings = $postgresql::server::default_connect_settings,
3333
Enum['present', 'absent'] $ensure = 'present',
3434
String $group = $postgresql::server::group,

‎manifests/server/extension.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
Optional[String[1]] $version = undef,
3030
Enum['present', 'absent'] $ensure = 'present',
3131
Optional[String[1]] $package_name = undef,
32-
Optional[Variant[String[1], Stdlib::Port]] $port = undef,
32+
Optional[Stdlib::Port] $port = undef,
3333
Hash $connect_settings = postgresql::default('default_connect_settings'),
3434
String[1] $database_resource_name = $database,
3535
) {

‎manifests/server/grant.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
Array[String[1],0] $object_arguments = [],
4242
String $psql_db = $postgresql::server::default_database,
4343
String $psql_user = $postgresql::server::user,
44-
Variant[String[1], Stdlib::Port] $port = $postgresql::server::port,
44+
Stdlib::Port $port = $postgresql::server::port,
4545
Boolean $onlyif_exists = false,
4646
Hash $connect_settings = $postgresql::server::default_connect_settings,
4747
Enum['present', 'absent'] $ensure = 'present',

‎manifests/server/grant_role.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
Enum['present', 'absent'] $ensure = 'present',
1414
String[1] $psql_db = $postgresql::server::default_database,
1515
String[1] $psql_user = $postgresql::server::user,
16-
Variant[String[1], Stdlib::Port] $port = $postgresql::server::port,
16+
Stdlib::Port $port = $postgresql::server::port,
1717
Hash $connect_settings = $postgresql::server::default_connect_settings,
1818
) {
1919
case $ensure {

‎manifests/server/instance/config.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848
String[1] $ip_mask_deny_postgres_user = $postgresql::server::ip_mask_deny_postgres_user,
4949
String[1] $ip_mask_allow_all_users = $postgresql::server::ip_mask_allow_all_users,
5050
Optional[Variant[String[1], Array[String[1]]]] $listen_addresses = $postgresql::server::listen_addresses,
51-
Variant[String[1], Stdlib::Port] $port = $postgresql::server::port,
51+
Stdlib::Port $port = $postgresql::server::port,
5252
Array[String[1]] $ipv4acls = $postgresql::server::ipv4acls,
5353
Array[String[1]] $ipv6acls = $postgresql::server::ipv6acls,
5454
Variant[String[1], Stdlib::Absolutepath] $pg_hba_conf_path = $postgresql::server::pg_hba_conf_path,

‎manifests/server/instance/late_initdb.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
String[1] $user = $postgresql::server::user,
1717
String[1] $group = $postgresql::server::group,
1818
Variant[String[1], Stdlib::Absolutepath] $psql_path = $postgresql::server::psql_path,
19-
Variant[String[1], Stdlib::Port] $port = $postgresql::server::port,
19+
Stdlib::Port $port = $postgresql::server::port,
2020
String[1] $module_workdir = $postgresql::server::module_workdir,
2121
) {
2222
# Set the defaults for the postgresql_psql resource

‎manifests/server/instance/passwd.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
String[1] $user = $postgresql::server::user,
1818
String[1] $group = $postgresql::server::group,
1919
Variant[String[1], Stdlib::Absolutepath] $psql_path = $postgresql::server::psql_path,
20-
Variant[String[1], Stdlib::Port] $port = $postgresql::server::port,
20+
Stdlib::Port $port = $postgresql::server::port,
2121
String[1] $database = $postgresql::server::default_database,
2222
String[1] $module_workdir = $postgresql::server::module_workdir,
2323
Optional[Variant[String[1], Sensitive[String[1]], Integer]] $postgres_password = $postgresql::server::postgres_password,

‎manifests/server/instance/service.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
Optional[String[1]] $service_provider = $postgresql::server::service_provider,
2626
String[1] $service_status = $postgresql::server::service_status,
2727
String[1] $user = $postgresql::server::user,
28-
Variant[String[1], Stdlib::Port] $port = $postgresql::server::port,
28+
Stdlib::Port $port = $postgresql::server::port,
2929
String[1] $default_database = $postgresql::server::default_database,
3030
Variant[String[1], Stdlib::Absolutepath] $psql_path = $postgresql::server::psql_path,
3131
Hash $connect_settings = $postgresql::server::default_connect_settings,

‎manifests/server/instance/systemd.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
# @param drop_in_ensure sets the Systemd drop-in file to present or absent
44
# @api private
55
define postgresql::server::instance::systemd (
6-
Variant[String[1], Stdlib::Port] $port,
6+
Stdlib::Port $port,
77
Stdlib::Absolutepath $datadir,
88
Optional[String[1]] $extra_systemd_config = undef,
99
String[1] $service_name = $name,

‎manifests/server/reassign_owned_by.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
String $new_role,
1414
String $db,
1515
String $psql_user = $postgresql::server::user,
16-
Variant[String[1], Stdlib::Port] $port = $postgresql::server::port,
16+
Stdlib::Port $port = $postgresql::server::port,
1717
Hash $connect_settings = $postgresql::server::default_connect_settings,
1818
) {
1919
$sql_command = "REASSIGN OWNED BY \"${old_role}\" TO \"${new_role}\""

‎manifests/server/role.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
Boolean $createdb = false,
3030
Boolean $createrole = false,
3131
String[1] $db = $postgresql::server::default_database,
32-
Optional[Variant[String[1], Stdlib::Port]] $port = undef,
32+
Optional[Stdlib::Port] $port = undef,
3333
Boolean $login = true,
3434
Boolean $inherit = true,
3535
Boolean $superuser = false,

‎manifests/server/table_grant.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
String[1] $db,
2020
String[1] $role,
2121
Optional[Enum['present', 'absent']] $ensure = undef,
22-
Optional[Variant[String[1], Stdlib::Port]] $port = undef,
22+
Optional[Stdlib::Port] $port = undef,
2323
Optional[String[1]] $psql_db = undef,
2424
Optional[String[1]] $psql_user = undef,
2525
Optional[Hash] $connect_settings = undef,

‎spec/acceptance/alternative_port_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
describe 'postgresql::server' do
88
it 'on an alternative port' do
99
pp = <<-MANIFEST
10-
class { 'postgresql::server': port => '55433', manage_selinux => true }
10+
class { 'postgresql::server': port => 55433, manage_selinux => true }
1111
MANIFEST
1212
if os[:family] == 'redhat' && os[:release].start_with?('8')
1313
apply_manifest(pp, expect_failures: false)

‎spec/defines/server/extension_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@
198198
extension: 'pg_repack',
199199
connect_settings: { 'PGHOST' => 'postgres-db-server',
200200
'DBVERSION' => '9.1',
201-
'PGPORT' => '1234' }
201+
'PGPORT' => 1234 }
202202
}
203203
end
204204

@@ -222,7 +222,7 @@
222222
extension: 'pg_repack',
223223
connect_settings: { 'PGHOST' => 'postgres-db-server',
224224
'DBVERSION' => '9.1',
225-
'PGPORT' => '1234' },
225+
'PGPORT' => 1234 },
226226
port: 5678
227227
}
228228
end

‎spec/defines/server/grant_role_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
let(:params) do
3232
super().merge(psql_db: 'postgres',
3333
psql_user: 'postgres',
34-
port: '5432')
34+
port: 5432)
3535
end
3636

3737
it {

‎templates/systemd-override.conf.epp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
<%- |
2-
Variant[String[1], Stdlib::Port] $port,
2+
Stdlib::Port $port,
33
Stdlib::Absolutepath $datadir,
44
Optional[String[1]] $extra_systemd_config,
55
| -%>

0 commit comments

Comments
 (0)