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

Fail explicitly when missing logfile permissions #666

Closed
mxsasha opened this issue Jul 12, 2022 · 0 comments
Closed

Fail explicitly when missing logfile permissions #666

mxsasha opened this issue Jul 12, 2022 · 0 comments
Assignees
@mxsasha
Milestone
Release 4.3

Comments

@mxsasha
Copy link
Collaborator

mxsasha commented Jul 12, 2022

If IRRD is started as root, configured to drop privileges, but the logfile only allows writing by root, it will start successfully, but processes will break as soon as they try to write a log line. Silently, as they can not write a log line explaining why they failed. IRRD should probably check these permissions and fail in a clear way.
@mxsasha mxsasha self-assigned this Jul 12, 2022
@mxsasha mxsasha added this to the Release 4.3 milestone Jul 12, 2022
mxsasha added a commit that referenced this issue Jan 12, 2023
@mxsasha
Fix #666 - Fail explicitly when missing logfile permissions
1b4735e
mxsasha added a commit that referenced this issue Jan 12, 2023
@mxsasha
Fix #666 - Fail explicitly when missing logfile permissions
599a1c7
mxsasha added a commit that referenced this issue Jan 12, 2023
@mxsasha
Fix #666 - Fail explicitly when missing logfile permissions
e34b6db
@tangledhelix tangledhelix mentioned this issue Apr 24, 2024
Closed
mxsasha added a commit that referenced this issue Apr 25, 2024
@mxsasha
Fix #937 - Fix incorrect log access check in setuid startup
e045dd0 
Introduced in #666
@mxsasha mxsasha mentioned this issue Apr 25, 2024
Merged
mxsasha added a commit that referenced this issue Apr 25, 2024
@mxsasha
Fix #937 - Fix incorrect log access check in setuid startup
55db11b 
Introduced in #666
mxsasha added a commit that referenced this issue Apr 25, 2024
@mxsasha
Fix #937 - Fix incorrect log access check in setuid startup
6213af3 
Introduced in #666
mergify bot pushed a commit that referenced this issue Apr 25, 2024
@mxsasha @mergify
Fix #937 - Fix incorrect log access check in setuid startup
5f953e9 
Introduced in #666

(cherry picked from commit 6213af3)
mergify bot pushed a commit that referenced this issue Apr 25, 2024
@mxsasha @mergify
Fix #937 - Fix incorrect log access check in setuid startup
7569561 
Introduced in #666

(cherry picked from commit 6213af3)
This was referenced Apr 25, 2024
Closed
Closed
mxsasha added a commit that referenced this issue Apr 25, 2024
@mxsasha
Fix #937 - Fix incorrect log access check in setuid startup
defce02 
Introduced in #666
@mxsasha mxsasha mentioned this issue Apr 25, 2024
Merged
mxsasha added a commit that referenced this issue Apr 25, 2024
@mxsasha
Fix #937 - Fix incorrect log access check in setuid startup
2f4ca06 
Introduced in #666
mergify bot pushed a commit that referenced this issue Apr 25, 2024
@mxsasha @mergify
Fix #937 - Fix incorrect log access check in setuid startup
6a012cc 
Introduced in #666

(cherry picked from commit 2f4ca06)
mergify bot pushed a commit that referenced this issue Apr 25, 2024
@mxsasha @mergify
Fix #937 - Fix incorrect log access check in setuid startup
aca1ae2 
Introduced in #666

(cherry picked from commit 2f4ca06)
This was referenced Apr 25, 2024
Merged
Merged
mxsasha pushed a commit that referenced this issue Apr 25, 2024
@mergify
[4.4.x] Fix #937 - Fix incorrect log access check in setuid startup (#…
3139574 
…943)

Introduced in #666

(cherry picked from commit 2f4ca06)
mxsasha added a commit that referenced this issue Apr 25, 2024
@mxsasha
Fix #937 - Fix incorrect log access check in setuid startup
968b0b9 
Introduced in #666

(cherry picked from commit 2f4ca06)
mxsasha pushed a commit that referenced this issue Apr 25, 2024
@mergify
[4.3.x] Fix #937 - Fix incorrect log access check in setuid startup (#…
81bcfe7 
…942)

Introduced in #666

(cherry picked from commit 2f4ca06)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mxsasha mxsasha

Labels
None yet
Projects
None yet
Milestone
Release 4.3
Development

No branches or pull requests
1 participant
@mxsasha