Lissy93 / AdGuardian-Term

🛡️ Terminal-based, real-time traffic monitoring and statistics for your AdGuard Home instance
https://adguardian.as93.net
MIT License
901 stars 37 forks source link

Long stop time? #6

Open papampi opened 1 year ago

papampi commented 1 year ago

Is it for me or does it take long time for it to stop after hitting "q" or "ctrl+c" If it runs for a period of time it takes so much long time to stop and after stop terminal shows some gibberish like this:

~$ 184;27M35;183;27M35;182;27M35;180;27M35;178;27M35;175;27M35;172;27M35;170;27M35;169;27M35;168;27M35;168;26M35;167;26M35;167;25M35;166;23M35;165;22M35;164;20M35;163;17M35;163;15M35;162;13M35;162;11M35;162;9M35;162;7M35;162;5M35;162;3M35;163;1M35;188;2M35;186;3M35;184;5M35;182;7M35;181;8M35;180;10M35;178;12M35;177;13M35;176;15M35;175;17M35;175;18M35;174;20M35;174;21M35;174;22M35;173;23M35;173;24M35;173;25M35;173;26M35;173;27M35;173;28M35;174;28M35;174;27M35;175;27M35;175;26M35;176;26M35;177;26M35;177;25M35;178;25M35;179;25M35;180;24M35;181;24M35;182;24M35;183;24M35;184;24M35;185;23M35;186;23M35;188;24M35;187;24M35;186;24M35;186;23M35;187;23M35;188;23M35;190;23M35;191;23M35;194;22M35;196;22M35;199;22M35;202;22M35;204;22M35;205;21M35;207;21M35;210;21M35;212;21M35;214;21M35;216;21M35;218;21M35;219;21M35;221;21M35;222;21M35;223;22M35;225;22M35;226;22M35;227;22M35;228;22M35;229;22M35;230;22M35;231;23M35;232;23M35;233;23M35;234;23M35;235;23M35;235;24M35;236;24M35;237;24M35;237;25M35;238;25M35;239;25M35;239;26M35;240;26M35;239;26M35;238;26M35;237;26M35;235;26M35;233;26M35;231;26M35;228;26M35;226;26M35;223;27M35;219;27M35;215;27M35;211;27M35;207;27M35;203;27M35;199;27M35;195;27M35;190;27M35;187;27M35;183;27M35;179;27M35;175;27M35;172;27M35;169;27M35;166;27M35;164;27M35;162;27M35;160;26M35;159;26M35;158;26M35;157;26M35;156;26M35;155;26M35;154;26M35;154;25M35;154;24M35;154;23M35;154;22M35;154;20M35;155;19M35;155;17M35;156;15M35;157;12M35;158;10M35;159;8M35;160;5M35;161;3M35;162;1M

Lissy93 commented 1 year ago

Yeah, I've seen this too. Only when executed directly, and it's worse if the update interval (ADGUARD_UPDATE_INTERVAL) is set to a higher number, or if the apps been running for a long while.

I think the waiting for shutdown signal needs to go in it's own thread, instead of being in the same one which triggers network requests. Right now, the ui.rs polls for input on 100ms frequency, and then notifies a waiter defined in main, that will call shutdown

I'm not sure if anyone else has any ideas too, but I'll try out a few things.

eddyg commented 6 months ago

Came here looking for why I couldn't stop adguardian (which I was running via docker run). 🙂

I ended up have to docker stop the container... and then my terminal filled with output like shown above.