Bug 6733 - problems switching from beta to production SQN servers
: problems switching from beta to production SQN servers
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: SqueezeNetwork Integration
: 7.0
: All All
: P2 normal (vote)
: ---
Assigned To: Brandon Black
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-01-23 01:30 UTC by Michael Herger
Modified: 2009-09-08 09:22 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 Michael Herger 2008-01-23 01:30:59 UTC
As the beta server didn't know my credentials, I wanted to switch back to production servers. But it failed and logged a few error messages:

[08-01-23 10:17:51.3756] Slim::Networking::SqueezeNetwork::_error (322) Unable to login to SN: Invalid username or password
[08-01-23 10:17:53.0008] Slim::Utils::Timers::checkTimers (200) Error: Timer failed: Can't call method "string" on an undefined value at /Users/mh/Documents/workspace/SC7.0/Slim/Networking/SqueezeNetwork.pm line 206.

It seems to be impossible to switch back to production if you don't have credentials on beta. Needed to restart SC to do the switch. It then worked ok on the production system.
Comment 1 Michael Herger 2008-01-23 01:36:21 UTC
The following patch might silence the string warning:

Index: /Users/mh/Documents/workspace/SC7.0/Slim/Networking/SqueezeNetwork.pm
===================================================================
--- /Users/mh/Documents/workspace/SC7.0/Slim/Networking/SqueezeNetwork.pm	(revision 16636)
+++ /Users/mh/Documents/workspace/SC7.0/Slim/Networking/SqueezeNetwork.pm	(working copy)
@@ -203,7 +203,7 @@
 	
 	# Return if we don't have any SN login information
 	if ( !$username || !$password ) {
-		my $error = $client->string('SQUEEZENETWORK_NO_LOGIN');
+		my $error = $client ? $client->string('SQUEEZENETWORK_NO_LOGIN') : string('SQUEEZENETWORK_NO_LOGIN');
 		$log->info( $error );
 		return $params{ecb}->( undef, $error );
 	}
Comment 2 Blackketter Dean 2008-01-27 16:41:09 UTC
Ping Brandon: is this properly assigned to you?
Comment 3 Brandon Black 2008-01-27 18:36:04 UTC
Michael's string warning patch has already been applied.  I've committed r16814 which makes the help for the beta switching preference not be a popup, I think people haven't been reading it.  A restart is always required to make that preference take effect in either direction.
Comment 4 Chris Owens 2008-03-07 09:03:40 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.