-
-
Notifications
You must be signed in to change notification settings - Fork 272
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
view=view_video mode is not working (requires ZM patch) #364
Comments
This is a ZoneMinder bug not ZmNinja - there are issues with the code - please post it in the ZM GitHub issue |
Strange, because browser plays it. |
yes, mobile view and browsers behave differently |
@abishai try updating your view_video.php with the latest 264 branch version |
This version works. Thanks! |
sounds good - can you also apply it to the VM you created for me? I will test other functions on monday |
Apply the file or bump git revision to the latest? |
nm, test server bumped to ZoneMinder/zoneminder@a2f782d |
thanks! |
/subj/ I'm forced to use direct path option.
I've noticed this issue https://github.com/pliablepixels/zmNinja/issues/275 but only workaround was implemented. Can't we dig to the roots of the problem? Direct linking without simple auth is a security flaw - video path can be easely guessed or bruted.
On nginx, I see this request when I open montage from timeline
10.0.1.5 - hakob [07/Nov/2016:21:28:04 +0300] "GET /index.php?view=view_video&eid=7359 HTTP/1.0" 200 15200648 "-" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) zmNinjaDesktop/0.1.0 Chrome/51.0.2704.103 Electron/1.2.5 Safari/537.36"
10.0.1.5 - hakob [07/Nov/2016:21:28:04 +0300] "GET /index.php?view=view_video&eid=7359 HTTP/1.0" 206 17539 "-" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) zmNinjaDesktop/0.1.0 Chrome/51.0.2704.103 Electron/1.2.5 Safari/537.36"
If I paste url to webbrowser (https://foo.bar/index.php?view=view_video&eid=7359) Firefox happily plays me using it's player.
PS. Sorry of I become too annoying, I'm creating 6th issue so far in 2 days, but I just migrated to h264 branch :P
The text was updated successfully, but these errors were encountered: