mherger wrote: 
> > Is this a yay or a nay for Paul's suggested one line change?
> 
> Depends on the change, of course. I haven't seen it yet.
> 

The change is to go from


Code:
--------------------
    
        } else {
                $clientid ||= $playername;
                $log->error(($clientid ? "$clientid: " : '') . "request not 
dispatchable!");
                Slim::Web::HTTP::closeHTTPSocket($context->{'httpClient'});
                return;
        }       
  
--------------------


to


Code:
--------------------
    
        } else {
                $clientid ||= $playername;
                $log->warn(($clientid ? "$clientid: " : '') . "request not 
dispatchable!");
                Slim::Web::HTTP::closeHTTPSocket($context->{'httpClient'});
                return;
        }       
  
--------------------



i.e. change "error" to "warn"

in Slim/Web/JSONRPC.pm - around line 486

I could send it as a pull request if that helps.



Paul Webster
http://dabdig.blogspot.com
Author Radio France (FIP etc) plugin
------------------------------------------------------------------------
Paul Webster's Profile: http://forums.slimdevices.com/member.php?userid=105
View this thread: http://forums.slimdevices.com/showthread.php?t=51929

_______________________________________________
plugins mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/plugins

Reply via email to