Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow to override all SecDefaultAction attributes #1486

Closed
wants to merge 1 commit into from
Closed

Allow to override all SecDefaultAction attributes #1486

wants to merge 1 commit into from

Conversation

sathieu
Copy link
Contributor

@sathieu sathieu commented Jun 17, 2016

No description provided.

This was referenced Jun 17, 2016
Closed
Closed
tphoney
tphoney reviewed Jun 21, 2016
View reviewed changes
spec/classes/mod/security_spec.rb
{ :audit_log_parts => "ABCDZ"
{
:audit_log_parts => "ABCDZ",
:secdefaultaction => "deny,status:406,nolog,auditlog"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs aligned

@tphoney
Copy link
Contributor

tphoney commented Jun 21, 2016

@sathieu this code looks pretty good, thanks for updating the readme as well. There is some minor alignment that needs corrected.

@DavidS DavidS mentioned this pull request Jun 30, 2016
Merged
@DavidS
Copy link
Contributor

DavidS commented Jun 30, 2016

Closing to prefer rebased and unified #1494

@DavidS DavidS closed this Jun 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@sathieu @tphoney @DavidS