Bug 4604 - no warning when can't create playlist because lack group write perm
: no warning when can't create playlist because lack group write perm
Status: CLOSED WORKSFORME
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.5.0
: PC Linux (other)
: P2 normal (vote)
: ---
Assigned To: Ross Levine
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-12-16 07:23 UTC by Judd Rogers
Modified: 2011-03-16 04:39 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Judd Rogers 2006-12-16 07:23:53 UTC
Gentoo Linux package installs slimserver as user/group slimserver (very nice thing to do).
However, my music is owned by my user id and the default group users.
While installing and setting up, I created a directory for playlists. 
I did not think about file access so left it as me/users.
Playlists did not work until I thought to check for write permission.
Once I made the playlist dir belong to group slimserv with group write permissions, everything began to work.
Prior to that, things just did not work: no errors or anything. Not lUser friendly! :-^

The issue is that the web interface never warned me about this.
I suspect there were warnings in the log but what non-programmer is going to look there?

Suggested fix:
1) When you make setting changes which add/change a path to a file or directory, confirm slimserver has read/write access.
2) If you don't have access, TELL the user (with a suggested solution like chgrp slimserver; chmod g+w file) (and the user must be in group slimserver to do the chgrp) via:
   a) pop-up window
   b) some visual indicator next to the text box containing the path to the file/directory
      (which would get you to a new window with the help required to fix the problem)

However you fix it, please remember: every error needs a visible warning. Either on the slimserver display or in the Currently Playing display on the web client. Logs are for programmers, not users. :-)
Comment 1 Chris Owens 2007-02-12 15:03:17 UTC
Ross, could you test this against slimserver 7.0?  It's not fixable under 6.5.x, but it should be a good test case for 7.0's improved error handling.  But, maybe it works already!
Comment 2 Ross Levine 2007-02-21 16:25:04 UTC
I agree what you're describing should not happen, however I'm not able to reproduce this here. When you say playlists did not work, what exactly do you mean? You weren't able to save new playlists using SlimServer, or older playlists will not play? 
Comment 3 Judd Rogers 2007-02-22 12:03:49 UTC
(In reply to comment #2)
> I agree what you're describing should not happen, however I'm not able to
> reproduce this here. When you say playlists did not work, what exactly do you
> mean? You weren't able to save new playlists using SlimServer, or older
> playlists will not play? 
> 
This was a new install. I could not save a new play list.
I might not even have been able to try to create a play list but my recollection is uncertain.
Comment 4 Ross Levine 2007-02-23 17:28:49 UTC
I just tried this again and I'm able to save a playlist without making changes to the permissions of the playlist directory. 

Could you please try reproducing this again? Or do you maybe have any other ideas as to what may have caused this, based on what I've tried so far I'd think I should be able to reproduce this problem fairly easily. 
Comment 5 Ross Levine 2007-03-16 15:50:04 UTC
I'm going to mark this bug works for me for now. Judd (or anyone else) if you're still experiencing this issue or it crops up again in the future please feel free to re-open the bug. 
Comment 6 Judd Rogers 2007-03-16 17:45:43 UTC
(In reply to comment #5)
> I'm going to mark this bug works for me for now. Judd (or anyone else) if
> you're still experiencing this issue or it crops up again in the future please
> feel free to re-open the bug. 
> 

OK, with the directory no longer owned by group slimserver, I can create playlists.
They are _not_ stored in the playlist directory but seem to be stored someplace
(in mysql?).

So, all is good. :-)
Comment 7 Ross Levine 2007-03-20 14:58:58 UTC
SlimServer will save the playlist to your playlist directory if and only if it has permission to do so. If it does not, or there is no playlist directory set, it will save the playlist to mysql and not create a playlist file on the disk. 
Comment 8 Chris Owens 2008-03-07 09:05:03 UTC
This bug is being closed since it was resolved for a version which is now released!  Please download the new version of SqueezeCenter (formerly SlimServer) at http://www.slimdevices.com/su_downloads.html

If you are still seeing this bug, please re-open it and we will consider it for a future release.