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

Feat: listen for unix signal #86

Merged
merged 2 commits into from
Jan 6, 2023

Conversation

dlhxzb
Copy link
Contributor

@dlhxzb dlhxzb commented Aug 4, 2022

When docker stop/restart, it would send SIGTERM and wait then SIGKILL.
Without listening for it we have to wait SIGTERM timeout here.
Fixes: #36

@rustdesk
Copy link
Owner

rustdesk commented Aug 4, 2022

Please remove unnessessary format change, your format is different from ours (default VSC rust anaylizer).

@dlhxzb
Copy link
Contributor Author

dlhxzb commented Aug 5, 2022

Is it possible that your VSC plug-in does not work at all 👀

use hbb_common::{
    log,
    rendezvous_proto::*,
    tokio::sync::{Mutex, RwLock},
    bytes::Bytes,  // No formatter would put `b***` here
    ResultType,
};

BTW either analyzer or cargo fmt, all used rustfmt as formatter

@rustdesk rustdesk merged commit d7b2060 into rustdesk:master Jan 6, 2023
@rustdesk
Copy link
Owner

rustdesk commented Jan 6, 2023

Sorry for this late PR merge, I put all my effort on rustdesk repo, and totally ignored this one.

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.

Server in the container does not respond to SIGINT/SIGTERM
2 participants