Only open camera when capturing about cam2ip HOT 7 OPEN

There is already notification for close here https://github.com/gen2brain/cam2ip/blob/master/handlers/mjpeg.go#L41 and for HTML on socket close can close the camera. My point is that if there is a need for e.g. many mjpeg clients, one can put some mjpeg-proxy in front of cam2ip, there are many of them, if lazy option doesn't work for many clients, user can disable it etc.
Just trying to not further complicate the code, for now everything is clear and simple, but I will consider also your hint when I start to work on this.

from cam2ip.

It seems like a nice addition. This should be done by handlers, not Read(), and if option enabled main should not initialize the camera. I think it should not be a big job but need to check first, not sure when. What do you think option should be called, lazy sounds ok?

from cam2ip.

"Deferred camera initialization" is the most descriptive way, but that doesn't fit well into a short CLI option, so "lazy" works. Kind of fun.

from cam2ip.

One thing to think about is, what happens when multiple clients are accessing the camera at once.

Would need to maintain a client counter and only release the camera when the client counter reaches zero.

from cam2ip.

Every handler will anyway check if camera is nil/null on every request and will initialize it if it is not, so one client closes tab/connection and it closes the camera, the other client on the next request will again initialize the camera. Needs to be tested but I would not want to complicate this much.

from cam2ip.

If somebody is viewing MJPEG stream they would not want to have the camera turned off underneath them by somebody capturing a single JPEG... This would result in a glitch in the video stream as the camera has to be reinitialized for the next MJPEG frame. I guess that's not the end of the world, but not perfect. Or is that not an issue for some other reason I'm not seeing? Oh well! It's your project, your call!

Thanks again for this project and thanks for being so responsive!

from cam2ip.

Maybe initialize as you suggest, and handle closing the camera by a ConnState hook ( https://golang.org/pkg/net/http/#ConnState ) that increments a counter on StateNew, decrements on StateClosed (maybe StateHijacked too) and closes the camera whenever the counter transitions to zero.

Again not a Go expert, just reading the docs...

from cam2ip.