{"action":"created","issue":{"url":"https://api.github.com/repos/ghoneycutt/puppet-module-pam/issues/97","labels_url":"https://api.github.com/repos/ghoneycutt/puppet-module-pam/issues/97/labels{/name}","comments_url":"https://api.github.com/repos/ghoneycutt/puppet-module-pam/issues/97/comments","events_url":"https://api.github.com/repos/ghoneycutt/puppet-module-pam/issues/97/events","html_url":"https://github.com/ghoneycutt/puppet-module-pam/pull/97","id":56414362,"number":97,"title":"EL 6 - Adding support for password-auth","user":{"login":"nbentoumi","id":6154423,"avatar_url":"https://avatars.githubusercontent.com/u/6154423?v=3","gravatar_id":"","url":"https://api.github.com/users/nbentoumi","html_url":"https://github.com/nbentoumi","followers_url":"https://api.github.com/users/nbentoumi/followers","following_url":"https://api.github.com/users/nbentoumi/following{/other_user}","gists_url":"https://api.github.com/users/nbentoumi/gists{/gist_id}","starred_url":"https://api.github.com/users/nbentoumi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nbentoumi/subscriptions","organizations_url":"https://api.github.com/users/nbentoumi/orgs","repos_url":"https://api.github.com/users/nbentoumi/repos","events_url":"https://api.github.com/users/nbentoumi/events{/privacy}","received_events_url":"https://api.github.com/users/nbentoumi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":22,"created_at":"2015-02-03T18:06:51Z","updated_at":"2015-11-04T11:49:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ghoneycutt/puppet-module-pam/pulls/97","html_url":"https://github.com/ghoneycutt/puppet-module-pam/pull/97","diff_url":"https://github.com/ghoneycutt/puppet-module-pam/pull/97.diff","patch_url":"https://github.com/ghoneycutt/puppet-module-pam/pull/97.patch"},"body":"Created another pull request EL 6 - Adding support for password-auth\r\n\r\nPam config for RHEL6 uses both system-auth and password-auth but the module only manages system-auth.\r\n\r\n\"The problem with /etc/pam.d/system-auth is that it contains modules that are not usable in remote configurations so remote services such as sshd, vsftpd now use /etc/pam.d/password-auth.\" https://access.redhat.com/site/solutions/45492.\r\n"},"comment":{"url":"https://api.github.com/repos/ghoneycutt/puppet-module-pam/issues/comments/153696840","html_url":"https://github.com/ghoneycutt/puppet-module-pam/pull/97#issuecomment-153696840","issue_url":"https://api.github.com/repos/ghoneycutt/puppet-module-pam/issues/97","id":153696840,"user":{"login":"Phil-Friderici","id":5038125,"avatar_url":"https://avatars.githubusercontent.com/u/5038125?v=3","gravatar_id":"","url":"https://api.github.com/users/Phil-Friderici","html_url":"https://github.com/Phil-Friderici","followers_url":"https://api.github.com/users/Phil-Friderici/followers","following_url":"https://api.github.com/users/Phil-Friderici/following{/other_user}","gists_url":"https://api.github.com/users/Phil-Friderici/gists{/gist_id}","starred_url":"https://api.github.com/users/Phil-Friderici/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Phil-Friderici/subscriptions","organizations_url":"https://api.github.com/users/Phil-Friderici/orgs","repos_url":"https://api.github.com/users/Phil-Friderici/repos","events_url":"https://api.github.com/users/Phil-Friderici/events{/privacy}","received_events_url":"https://api.github.com/users/Phil-Friderici/received_events","type":"User","site_admin":false},"created_at":"2015-11-04T11:49:39Z","updated_at":"2015-11-04T11:49:39Z","body":"enhanced the readability of the comment that @nbentoumi added on Sep 14:\r\n\r\n> Sorry, was not aware about this process, in my last commit, I have added the code below as suggested\r\n\r\n(added the lines 344 to 358 to init_spec.rb)\r\n>```\r\n            if v[:osfamily] == 'RedHat'\r\n              if (v[:release] == '6' or v[:release] == '7')\r\n                  it {\r\n                    should contain_file('pam_password_auth_ac').with({\r\n                      'ensure' => 'file',\r\n                      'path'   => '/etc/pam.d/password-auth-ac',\r\n                      'owner'  => 'root',\r\n                      'group'  => 'root',\r\n                      'mode'   => '0644',\r\n                    })\r\n                  }\r\n                  pam_password_auth_ac_fixture = File.read(fixtures(\"pam_password_auth_ac.#{check}.#{k}\"))\r\n                  it { should contain_file('pam_password_auth_ac').with_content(pam_password_auth_ac_fixture) }\r\n              end\r\n            end\r\n```\r\n> added to $default_pam_password_password_lines the line below on el7 with vas (found this when\r\nadded the spec test above)\r\n\r\n(added the line 368 and 428 to init.pp)\r\n>`    'password requisite pam_pwquality.so try_first_pass local_users_only retry=3 authtok_type=',`\r\n"}}