-
Notifications
You must be signed in to change notification settings - Fork 22
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.
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
Fixes #10962: Add ssl based authentication to policies #1157
Fixes #10962: Add ssl based authentication to policies #1157
Conversation
@@ -0,0 +1,82 @@ | |||
##################################################################################### | |||
# Copyright 2014 Normation SAS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2017
@@ -42,7 +53,6 @@ along with this program. If not, see <http://www.gnu.org/licenses/>. | |||
<INCLUDED>false</INCLUDED> | |||
</TML> | |||
<TML name="nodeslist.json"> | |||
<OUTPATH>distributePolicy/1.0/nodeslist.json</OUTPATH> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this the default value?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
|
||
rudder_apache_acl_repaired:: | ||
|
||
"${paths.path[service]}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not service_reload("${apache_service}")
?
Commit modified |
bf2be31
to
eea0c23
Compare
|
||
rudder_apache_acl_repaired:: | ||
"any" usebundle => rudder_common_report("DistributePolicy", "log_info", "&TRACKINGKEY&", "Configure apache ACL", "None", "Apache has been reloaded successfully"), | ||
ifvarclass => "${apache_service}_repaired" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
service_reload_${apache_service}_repaired
?
Commit modified |
eea0c23
to
6b02efc
Compare
"any" usebundle => rudder_common_report("DistributePolicy", "result_error", "&TRACKINGKEY&", "Configure apache ACL", "None", "Apache ACLs could not be edited"); | ||
|
||
rudder_apache_acl_repaired:: | ||
"any" usebundle => rudder_common_report("DistributePolicy", "log_info", "&TRACKINGKEY&", "Configure apache ACL", "None", "Apache has been reloaded successfully"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
log_repaired ?
Commit modified |
6b02efc
to
69382e9
Compare
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/10962