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

Killing FTL via the FTL API does not fully kill FTL #379

Closed
3 tasks done
AzureMarker opened this issue Sep 9, 2018 · 1 comment
Closed
3 tasks done

Killing FTL via the FTL API does not fully kill FTL #379

AzureMarker opened this issue Sep 9, 2018 · 1 comment

Comments

@AzureMarker
Copy link
Contributor

In raising this issue, I confirm the following:

  • I have read and understood the contributors guide.
  • The issue I am reporting can be replicated
  • The issue I am reporting isn't a duplicate

How familiar are you with the codebase?:
10


[BUG | ISSUE] Expected Behaviour:
Running echo ">kill" | nc localhost 4711 will kill FTL.

[BUG | ISSUE] Actual Behaviour:
Running the above command does not kill all of FTL.

[BUG | ISSUE] Steps to reproduce:

  • Start FTL
  • Run pstree pihole and see all threads running
pihole-FTL─┬─{database}
           ├─{housekeeper}
           ├─{socket listener}
           ├─{telnet-IPv4}
           └─{telnet-IPv6}
  • Run echo ">kill" | nc 127.0.0.1 4711
  • Run pstree pihole and see that the housekeeper and database threads have died
pihole-FTL─┬─{socket listener}
           ├─{telnet-IPv4}
           └─{telnet-IPv6}
  • Run the same commands again and see that the telnet-IPv4 thread has died
pihole-FTL─┬─{socket listener}
           └─{telnet-IPv6}
  • Run echo ">kill" | nc ::1 4711 to kill FTL over IPv6
  • Run pstree pihole and see that the telnet-IPv6 thread has died
pihole-FTL───{socket listener}
  • (Optional fun) Send the >kill command over the FTL socket to kill the socket thread. Now pstree pihole shows:
pihole-FTL
  • No more ways to kill FTL over the FTL API, because there are no more FTL API listeners! Note: DNS resolution is still available.

Log file output [if available]

[2018-09-09 14:11:36.252] FTL killed by client ID: 21
[2018-09-09 14:12:15.569] FTL killed by client ID: 20
[2018-09-09 14:13:42.052] FTL killed by client ID: 18

Device specifics

Hardware Type: Tinkerboard
OS: Raspbian

This template was created based on the work of udemy-dl.

@AzureMarker
Copy link
Contributor Author

Fixed by #395, which removes the kill command.

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

No branches or pull requests

2 participants