Skip to content

Commit 3276d3b

Browse files
committed
fix(debian-ish): root owns the logs
1 parent 3d5723d commit 3276d3b

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

openvpn/config.sls

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ openvpn_{{ type }}_{{ name }}_log_file:
169169
- name: {{ config.log }}
170170
- makedirs: True
171171
- replace: False
172-
{{ _permissions(640) }}
172+
{{ _permissions(640, map.log_user) }}
173173
- require_in:
174174
{%- if map.multi_services %}
175175
- service: openvpn_{{name}}_service
@@ -185,7 +185,7 @@ openvpn_{{ type }}_{{ name }}_log_file_append:
185185
- name: {{ config.log_append }}
186186
- makedirs: True
187187
- replace: False
188-
{{ _permissions(640) }}
188+
{{ _permissions(640, map.log_user) }}
189189
- require_in:
190190
{%- if map.multi_services %}
191191
- service: openvpn_{{name}}_service

openvpn/defaults.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ openvpn:
77
external_repo_supported: []
88
external_repo_version: stable
99
group: nobody
10+
# None, will default to 'user'
11+
log_user:
1012
multi_services: False
1113
pkgs: ['openvpn']
1214
service: openvpn

openvpn/osfamilymap.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ Arch:
44
- easy-rsa
55
Debian:
66
group: nogroup
7+
log_user: root
78
RedHat:
89
pkgs:
910
- openvpn

0 commit comments

Comments
 (0)