Merge pull request #12 from hfm/user

Set home directory and default shell for proxysql user
hfm · Sep 14, 2018 · db90fd7 · db90fd7
2 parents c7a8b8c + 8632d2e
commit db90fd7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/manifests/install.pp b/manifests/install.pp
@@ -6,9 +6,12 @@
 
   if $proxysql::owner != 'root' {
     user { $proxysql::owner:
-      ensure => present,
-      gid    => $proxysql::group,
-      system => true,
+      ensure  => present,
+      comment => 'ProxySQL',
+      gid     => $proxysql::group,
+      home    => $proxysql::datadir,
+      shell   => '/bin/false',
+      system  => true,
     }
   }
 

diff --git a/spec/acceptance/proxysql_spec.rb b/spec/acceptance/proxysql_spec.rb
@@ -45,10 +45,13 @@
 
   describe user('proxysql') do
     it { is_expected.to exist }
+    it { is_expected.to have_home_directory '/var/lib/proxysql' }
+    it { is_expected.to have_login_shell '/bin/false' }
+    it { is_expected.to belong_to_primary_group 'proxysql' }
   end
 
-  describe user('proxysql') do
-    it { is_expected.to belong_to_primary_group 'proxysql' }
+  describe group('proxysql') do
+    it { is_expected.to exist }
   end
 
   describe file('/etc/proxysql.cnf') do