-
Notifications
You must be signed in to change notification settings - Fork 71
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
php 7.4 #39
Comments
Looks like this error is coming up because of the query failing. I should probably have a check to validate the query completed successfully, but this error can only really occur if something went wrong on the postgres side of the house. |
not sure if it's within pgsql. https://imgur.com/N2paCff here is the exact error, when you open the player, and click play :) any advice will be much appreciated ;) also would highly recommend the streamkey for users to be created something like user_random_uuid and look like this (for example)
|
I'm not sure I will be able to help much any time soon though, this project is more or less on indefinite hiatus and I don't have any current plans to update my server to 7.4 to test against. According to the error itself in the nginx logs, the query is failing and returning a value that the rest of the function isn't expecting, but I can't validate if that is the case as it's not happening on my end. I would recommend going to the main config, setting debug to 1 and start adding breakpoints in the function itself where it is failing to validate each step: https://github.com/Fenrirthviti/stream-site/blob/master/lib/user.class.php#L458 The problem with that approach to stream keys is that nginx-rtmp (and http-flv by extension) doesn't allow for different ingest/playback URLs. That means anyone viewing will be able to see what your stream key is and stream as you. I would have to build some kind of custom loopback system to split out ingest from playback, which is a bit outside my current experience and skillset, so I went with the best option I could to still have some form of authentication. The current solution allows you to pass and parse URL params (which can be anything, I left it as key= for simplicity) which are not required for playback. |
tried debugging but not sure why you are suggesting it, but it's already set to 1, and the database returns results as normal - https://imgur.com/a/QXlMcp2 sending mails works flawlessly - tried with my smtp server - mail delivers, the confirmation URL is valid, and enabling account is fine. what concerns me most, is that there is no reason whatsoever to get those errors. if i start playing the stream, with some media player, everything is just fine. btw, i don't mind giving you access to my system, if you are interested to see, but the only reason i could be that i'm using openresty. if you strongly suggest to compile nginx, will do so. overall openresty is quite handy when you need all-in-one custom libs/support.
|
Debug to 1 turns on global PHP error reporting directly in the page. If everything is working and you are only seeing this warning, set debug to 0 and see if it goes away. Debug shouldn't be left at 1 unless you're trying to diagnose an actual problem, as it makes all errors and warnings extremely verbose. The "error" itself is only a php warning message due to a change on how php7.4 handles certain edge case conditions, it's not a true error. It shouldn't be enough to halt the actual page functionality from happening. Is something not working, or is it just the warning message that has you concerned? If there is something specific that is not working properly that might help me understand where the problem might be. |
yes, the problem is that the video is not playable .. https://imgur.com/a/LsUaTW2 check those 2 screenshot one is popout player on console you can see that there is ffmpeg process, which actually is making an mp4 file. |
Are you using nginx-rtmp or nginx-http-flv? If you're using rtmp, you'll need to make sure video.js is configured for rtmp playback and that flash is explicitly allowed. This will only work for the next few months, as everyone is pulling support for flash in January 2020, which is why I have most of the site migrated to nginx-http-flv. |
it's compiled with both modules. where should i configure the video.js? |
this is when the stream is on, and the player trying to show something... |
hey again.
since migrating to php-fpm 7.4 i'm getting the following error:
double checked about missing modules, but not that issue. could you be so kind and let me know, what could go wrong
The text was updated successfully, but these errors were encountered: