After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 487203 - DAAP password authenication problems when accessing a DAAP server
DAAP password authenication problems when accessing a DAAP server
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: DAAP
git master
Other All
: Normal normal
: 1.x
Assigned To: Banshee Maintainers
Banshee Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-10-16 16:12 UTC by Trevor Coverley
Modified: 2010-01-26 00:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Use default content codes for password protected servers (947 bytes, patch)
2008-09-29 09:42 UTC, Félix Velasco
none Details | Review
Only use default codes for password protected servers, no for net failures (1.06 KB, patch)
2008-09-29 21:34 UTC, Félix Velasco
reviewed Details | Review
Delay retrieval of content-codes until credentials are provided (916 bytes, patch)
2009-01-13 20:49 UTC, Félix Velasco
committed Details | Review

Description Trevor Coverley 2007-10-16 16:12:11 UTC
Please describe the problem:
I am running a DAAP server (not iTunes), Firefly Media Server v 1671.  

When I have the DAAP server setup to advertise the Library password protected, Banshee can see the share and it has the lock icon on it.  When I try to connect to it, I DON'T get prompted for the password.  

If I set up the Firefly Server without password authenication and have Banshee try to connect, Banshee PROMPTS me for a password, in which case I leave the password field blank, press enter and I can connect to the share.

The problem seems to reside in that Banshee won't prompt me to enter the share password when a share password exists.

Steps to reproduce:
1. The problem is reproduced when trying to connect to a password protected DAAP server
2. 
3. 


Actual results:
You never get prompted to enter the password and get this error:
- The provided login credentials are invalid
- The login process was canceled
- Too many users are connected to this share
- The music share is hosted by iTunes7

End result is that you can not connect to the share.

Expected results:
I would expect to get prompted to enter the DAAP server share password and have access to see the share.

Does this happen every time?
Yes

Other information:
On a Windows PC using Itunes I can connect to the Firefly server with no issues when password authentication is turned on or off.
Comment 1 Trevor Coverley 2007-10-16 16:58:18 UTC
Some additional info.  I installed rhythmbox(0.11.2) on the same box as Banshee is installed, password protected the Music on the Firefly server.  Rhythmbox prompted me to enter the share password and I was able to connect to the DAAP share after entering the password.
Comment 2 Félix Velasco 2008-09-29 09:42:35 UTC
Created attachment 119567 [details] [review]
Use default content codes for password protected servers
Comment 3 Félix Velasco 2008-09-29 21:34:10 UTC
Created attachment 119616 [details] [review]
Only use default codes for password protected servers, no for net failures
Comment 4 Bertrand Lorentz 2008-10-08 19:35:19 UTC
Changing version, as it seems that both the issue and the patch apply to trunk.
Comment 5 Gabriel Burt 2009-01-13 17:01:39 UTC
Félix, the real fix here would be to authenticate, no?  Setting the bag to the Default whenever we connect to a pwd-protected share sounds like a recipe for strange failures down the road.
Comment 6 Félix Velasco 2009-01-13 17:44:44 UTC
Well, the code to authenticate is already in banshee :) The problem is that we try to retrieve the content-codes, that are password-protected, before we ask the credentials to the user.

Btw, I've seen a forgot attaching a newer version of the patch. It just re-throws the exception if it's other kind of problem, only "bypassing" the authentication troubles.
Comment 7 Gabriel Burt 2009-01-13 18:18:06 UTC
Well, we should delay getting the content codes until we've authenticated, then. 
Comment 8 Félix Velasco 2009-01-13 20:22:33 UTC
mmm I think you're right. I'm working on it, will post a patch soon.
Comment 9 Félix Velasco 2009-01-13 20:49:17 UTC
Created attachment 126374 [details] [review]
Delay retrieval of content-codes until credentials are provided

I was trying something far more complicated, until I realized this one-liner works perfectly.
Comment 10 Gabriel Burt 2009-01-13 21:26:10 UTC
Much better.  Can we do it right after the Fetch ("/login") call though?  That makes the most sense to me - login, then see what kinda stuff it supports, then get the tracks etc..
Comment 11 Félix Velasco 2009-01-14 09:12:36 UTC
It seems iTunes works the other way (see http://www.tapjam.net/daap/ , chapter IV. Client/Server conversation overview). First retrieve server-info, then content-codes, then login.

I agree it's weird sending credentials before we are properly logging in, but at least firefly needs them for content-codes (not for server-info, it seems).
Comment 12 Brett Ussher 2009-08-02 20:01:23 UTC
I just added the patch posted by Félix Velasco to the source files in the Ubuntu repository for Jaunty (v.1.4.3) and it solved this issue.

Thanks, Félix!

Brett Ussher
Comment 13 Gabriel Burt 2010-01-26 00:43:19 UTC
Comment on attachment 126374 [details] [review]
Delay retrieval of content-codes until credentials are provided

Committed, thanks Felix.