Barracuda09 / SATPI

SATIP server for linux, suitable to run on an Raspberry Pi, Vu+, GigaBlue or any other linux box. currently supporting DVB-S/S2/T/C and transform DVB-S requests to DVB-C
http://barracuda09.github.io/SATPI
GNU General Public License v2.0
154 stars 32 forks source link

Bug with the childPIPE, subprocess not killed #77

Closed Barracuda09 closed 5 years ago

Barracuda09 commented 5 years ago

Hi @Barracuda09 ,

I found a SERIOUS bug with the childPIPE !!!

Please, check to kill the subprocess every time you close the stream. Thank tou!

Originally posted by @lars18th in https://github.com/Barracuda09/SATPI/issues/69#issuecomment-550274249

lars18th commented 5 years ago

Hi @Barracuda09 ,

This problem is not fixed and it continues. I can't do futher tests until this is resolved, as the server collapses with a lot of child process. So testing is difficult with last commit.

I hope you can target it soon. Thank you!

Barracuda09 commented 5 years ago

Hi @lars18th

Do you have an example using wget to read from something. I can not seem to get a use case.

lars18th commented 5 years ago

Do you have an example using wget to read from something. I can not seem to get a use case.

You can use the well-kown udproxy or similar UDP-2-HTTP relay : https://github.com/pcherenkov/udpxy So you can convert any multicast to HTTP... then you can receive the stream with exec=wget -qO- http://udpxy_ip:5056/udp/239.0.0.1:12345.

Regards.

Barracuda09 commented 5 years ago

This should be fixed now.

exec=wget -qO- http://udpxy_ip:5056/udp/239.0.0.1:12345

should be something like this (I think):

exec=wget%20-qO-%20http:%2F%2Fudpxy_ip:5056%2Fudp%2F239.0.0.1:12345
lars18th commented 5 years ago

Thank you @Barracuda09 ,

The issue is fixed with commit https://github.com/Barracuda09/SATPI/commit/6257b4b3e0b6f82f6b289129525a36f23fca992a

So, please close it. :wink:

Barracuda09 commented 5 years ago

Thanks, and as requested by @lars18th I will close this-one.