Project

General

Profile

Bug #2921

Improvements to PVR after last changes

Added by Rafal Kupiec over 9 years ago. Updated over 9 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
Category:
PVR / DVR
Target version:
-
Start date:
2015-06-04
Due date:
% Done:

100%

Estimated time:
Found in version:
4.1-118
Affected Versions:

Description

Last time, connection limit has been introduced to PVR. After this change I notice few thing that could be improved IMHO:

1) Some users might want to not limit DVR sessions, so it might be a good idea, to implement another check box in Access Entries, to tell TVH if connection limit should be applied to DVR too.
2) When connection limit is exhausted, recording fails with "File not found". This message says nothing about connection limit and should be changed.
3) Users without "All DVR" right set, should not see recordings scheduled (owned) by other users. I think that "Video Recorder" and "HTSP DVR" should allow owner of recording to play/download/remove it, but he should not see recordings owned by others. When "All DVR" right is granted, then the user should be able to see and playback recordings owned by others, and only with "All DVR (rw)" granted, he should be able to delete recordings owned by everyone.

History

#1

Updated by Jaroslav Kysela over 9 years ago

Implemented 1) and 2), but 3) should work as you described.

Are you sure that you have ACLs set correctly ? Note that the rights are "merged" also for the network ranges, so if you set "All DVR" for some IP range, the users connection from this range will have this permission, too. Anyway, "--trace access" should print the actual priviledges to check if the problem is in the condition in the code.

#2

Updated by Rafal Kupiec over 9 years ago

I have few users with the same subnet defined. That might be the reason why it is not working as described.
What is the reason to merge privileges for network range?

#3

Updated by Jaroslav Kysela over 9 years ago

I meant "anonymous user ranges" (users with usernames "*"). I think that it's logical to authentizate through IP or username/password. It would be a bit inconsistent that an IP authentization (anonymous) might have better priviledges than username/password authentization, wouldn't ?

#4

Updated by Rafal Kupiec over 9 years ago

Sounds good, but actually I got troubles with testing this TBH.
When I schedule recording as administrator via webui, then it records fine. When I schedule recording from Kodi as a regular user, then it fails with "missing file"...

#5

Updated by Jaroslav Kysela over 9 years ago

You should see the reason in logs.

#6

Updated by Rafal Kupiec over 9 years ago

Indeed:

I have set Conenction Limit Type to Streaming and Limited connection to 1.
When it fails with "missing fale" the logs says:

2015-06-05 22:05:18.087 [ ERROR]:dvr: multiple connections are not allowed for user 'belliash' from 'belliash' (limit 1, streaming 1, active DVR 1)

so point 2 is still not implemented, and looks like point 1 is buggy :(

#7

Updated by Jaroslav Kysela over 9 years ago

Both issues are fixed now. v4.1-165-gf101b85

#8

Updated by Rafal Kupiec over 9 years ago

Jaroslav Kysela wrote:

Both issues are fixed now. v4.1-165-gf101b85

Now fails with "User limit reached" error and after setting limit to streaming only then it also fails with the following error in log:

2015-06-05 22:33:03.601 [ ERROR]:dvr: multiple connections are not allowed for user 'belliash' from 'belliash' (limit 1, dvr limit 0, active DVR 1, streaming 1)

#9

Updated by Rafal Kupiec over 9 years ago

Rafal Kupiec wrote:

Jaroslav Kysela wrote:

Both issues are fixed now. v4.1-165-gf101b85

Now fails with "User limit reached" error and after setting limit to streaming only then it also fails with the following error in log:

2015-06-05 22:33:03.601 [ ERROR]:dvr: multiple connections are not allowed for user 'belliash' from 'belliash' (limit 1, dvr limit 0, active DVR 1, streaming 1)

Also, I think this is bad conception, because ppl who has limited both streaming and dvr, should be still able to launch kodi and begin recording... they are just connected to TVH, but not streaming/recording anything

#10

Updated by Jaroslav Kysela over 9 years ago

Another thinko fixed in 166.

#11

Updated by Rafal Kupiec over 9 years ago

Jaroslav Kysela wrote:

Another thinko fixed in 166.

still no help

#12

Updated by Jaroslav Kysela over 9 years ago

  • Status changed from New to Fixed
  • % Done changed from 0 to 100

Applied in changeset commit:tvheadend|5cc371720334e889a3879f76ce6cf3c664b85709.

Also available in: Atom PDF