Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@oeiber
Copy link
Contributor

@oeiber oeiber commented Nov 11, 2025

  • modified startup.sh to ensure node process runs with pid 1
  • modified server.ts to handle SIGTERM

- modified startup.sh to ensure node process runs with pid 1
- modified server.ts to handle SIGTERM
@RoelVB
Copy link
Collaborator

RoelVB commented Nov 12, 2025

Thank you for the PR. Unfortunately it doesn't compile (the SIGINT listener doesn't close anymore).
Also... I don't like that that the PR changes 24 lines of code, while it should only add 4 lines and change one. All the other changes just change the coding style (make it inconsistent with the rest of the code)

@oeiber
Copy link
Contributor Author

oeiber commented Nov 12, 2025

Sorry for that. Now it should work.

@RoelVB
Copy link
Collaborator

RoelVB commented Nov 14, 2025

Thank you very much

@RoelVB RoelVB merged commit 502aa2f into SMTP2Graph:main Nov 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants