Skip to content

Commit

Permalink
Merge pull request #39 from GutierrezJeremy/v2.2.0
Browse files Browse the repository at this point in the history
Replace legacy facts with modern facts
  • Loading branch information
fraenki committed Jan 23, 2023
2 parents 6abb1cd + e9e8faa commit 8073088
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ You're encouraged to define your configuration using Hiera, especially if you pl
UseReverseDNS: 'off'
DefaultRoot: '~ !'
DefaultChdir: '/var/ftp'
ServerName: '%{::fqdn}'
ServerName: '%{facts.networking.fqdn}'
Port: '21'
PassivePorts: '49152 65534'
TransferLog: 'NONE'
Expand All @@ -142,8 +142,8 @@ You're encouraged to define your configuration using Hiera, especially if you pl
TLSOptions: 'NoCertRequest'
TLSCipherSuite: 'ALL:!ADH:!EXPORT56:RC4+RSA:+HIGH:+MEDIUM:+LOW:+SSLv2:+EXP'
TLSVerifyClient: 'off'
TLSRSACertificateFile: '/etc/ssl/%{::fqdn}.crt'
TLSRSACertificateKeyFile: '/etc/ssl/%{::fqdn}.key'
TLSRSACertificateFile: '/etc/ssl/%{facts.networking.fqdn}.crt'
TLSRSACertificateKeyFile: '/etc/ssl/%{facts.networking.fqdn}.key'
TLSLog: '/var/log/proftpd/tls.log'
Global:
RequireValidShell: 'off'
Expand Down
4 changes: 2 additions & 2 deletions data/common.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,9 @@ proftpd::service_name: 'proftpd'
# Minimal default config, should work on most systems
proftpd::default_options:
'ROOT':
'ServerName': "ProFTPD server %{::fqdn}"
'ServerName': "ProFTPD server %{facts.networking.fqdn}"
'ServerIdent': 'on "FTP Server ready."'
'ServerAdmin': "root@%{::domain}"
'ServerAdmin': "root@%{fact.networking.domain}"
'DefaultServer': 'on'
'DefaultRoot': '~ !adm'
'AuthPAMConfig': 'proftpd'
Expand Down

0 comments on commit 8073088

Please sign in to comment.