when I said we should escape any problematic strings before they got to the redis slave, I meant in the redis stub (i.e. before pushing onto the redis queue we should escape any spaces). Having server.go explicitly check for spaces just because redis.go can't handle them makes no sense; what about cmdline.go? what about any other UIs we might add in the future that might use redis? 

I have no problem with server.go doing some validation and returning some errors and so forth, but this is the wrong layer to handle redis-related escaping. 

(also as a very very nit-picky thing, It "returns errors", not It "return errors" :)