Update rhel_system_auth.erb #120

Merged
merged 1 commit into from Jun 9, 2016

Conversation

Projects
None yet
4 participants
@phillym
Contributor

phillym commented Jun 7, 2016

Adding os-hardening root namespace to node reference for /etc/pam.d/system-auth-ac
Was testing your new Beta as previous versions break with the latest Chef client included in AWS Opsworks. Was getting an "undefined method `[]' for nil:NilClass" error on line 26 of rhel_system_auth.erb
This change has worked on our systems, looks like just an oversight.

Update rhel_system_auth.erb
Adding os-hardening root namespace to node reference for  /etc/pam.d/system-auth-ac
Was testing your new Beta as previous versions break with the latest Chef client included in AWS Opsworks. Was getting an "undefined method `[]' for nil:NilClass" error on line 26 of rhel_system_auth.erb
This change has worked on our systems, looks like just an oversight.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 7, 2016

Coverage Status

Changes Unknown when pulling ef41a4b on phillym:patch-1 into * on dev-sec:master*.

coveralls commented Jun 7, 2016

Coverage Status

Changes Unknown when pulling ef41a4b on phillym:patch-1 into * on dev-sec:master*.

@rljohnsn

This comment has been minimized.

Show comment
Hide comment
@rljohnsn

rljohnsn Jun 8, 2016

Contributor

Was just about to submit the same pull request.

Contributor

rljohnsn commented Jun 8, 2016

Was just about to submit the same pull request.

@atomic111

This comment has been minimized.

Show comment
Hide comment
@atomic111

atomic111 Jun 9, 2016

Member

thanks for your work!!!

Member

atomic111 commented Jun 9, 2016

thanks for your work!!!

@atomic111 atomic111 merged commit a39ccdf into dev-sec:master Jun 9, 2016

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
coverage/coveralls First build on master at 100.0%
Details

rollbrettler pushed a commit to rollbrettler/chef-os-hardening that referenced this pull request Sep 16, 2016

Merge pull request #120 from dev-sec/chris-rock/fedora
activate fedora integration tests in travis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment