MaximeMichaud / KVS-install

Automatic Linux Installation for Kernel Video Sharing
https://forum.kernel-video-sharing.com/topic/3808-automatic-installation-on-linux-for-kernel-video-sharing/
GNU General Public License v3.0
11 stars 6 forks source link

Fixing the KVS.conf video page issue #10

Closed yassinbahri closed 3 months ago

yassinbahri commented 5 months ago

This part of the configuration is obsolete now and will cause issues in NGINX loading the correct page when you go to a video:

rewrite ^/videos/([0-9]+)/([^/]+)/$      /view_video.php?id=$1&dir=$2 last;
rewrite ^/videos/([^/]+)/$               /view_video.php?dir=$1 last;

it should be like this instead

rewrite ^/video/([0-9]+)/([^/]+)/$      /view_video.php?id=$1&dir=$2 last;
rewrite ^/video/([^/]+)/$               /view_video.php?dir=$1 last;

I have found this out while configuring KVS on NGINX with Debian 11. Please fix this ASAP so that non technical people wouldn't be blocked.

MaximeMichaud commented 5 months ago

Hi, thank you, this has already been fixed yesterday. Now we are taking the KVS NGINX config directly from the archive. The change from videos to video was not present in the changelog because it is considered a theme change by the dev team and not a core change. I appreciate the heads-up, and in case Memcached is not on your installation, you can do the following: apt install memcached -y && sed -i 's/-m 64/-m 256/' /etc/memcached.conf && systemctl restart memcached

https://github.com/MaximeMichaud/KVS-install/commit/9b1b9120443a4f52296bb97c164384fb061c96a9#diff-c5588171a99c920100cf46c341cd3d14f68375776defdeacb73530401664b540R380