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.

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

Add support for daemonization #440

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

skrashevich
Copy link
Contributor

@skrashevich skrashevich commented May 21, 2023

This pull request introduces several enhancements and updates to the project. The major additions are as follows:

  • Daemonization Support: Added functionality to run the program as a daemon with the -d flag. This allows the program to run as a background process.
  • Dropping privileges: Introduced a flag to specify the user that the program should run as. For example, go2rtc might need root permissions in order to bind to a network port number lower than 1024 (as is the case in many Unix-like systems), but it does not need those permissions to handle incoming network connections once the port is open.
  • Signal Handling: Added signal handlers for SIGQUIT, SIGTERM, SIGSEGV, and SIGHUP. This improves the program's ability to handle different types of system signals and makes the program more robust. Additionally, the signal notifications in the shell package have been updated to include SIGQUIT and SIGSEGV.

All these changes apply only to unix-like systems. for windows stubs are implemented

@skrashevich skrashevich marked this pull request as ready for review May 21, 2023 16:38
@AlexxIT
Copy link
Owner

AlexxIT commented Jun 29, 2023

Lots of code. It needs to be carefully analyzed all of it.

@skrashevich
Copy link
Contributor Author

Lots of code. It needs to be carefully analyzed all of it.

The Way of the Maintainer :)

@skrashevich
Copy link
Contributor Author

Up 🙄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants