Removed "params" validation, content validation should be done by user application

Author: r3code

pyjsonrpclite/jsonrpc.py

@@ -135,22 +135,7 @@ def SubValidateMethod(jsondict):
             if not SubHasMethod(jsondict):
                 raise JsonRpcException('No "method" field')
             if not SubIsMethodCorrect(jsondict):
-                raise JsonRpcException('Invalid "method" field value')  
-                
-        def SubValidateParams(params):
-            '''Checks if params not empty. Raises `JsonRpcException` if invalid. 
-            Params: 
-                params -- {scalar|array|object} value of jsondict['params'] 
-            '''  
-            if params is None:
-                return True
-            if isinstance(params, basestring):
-                return len(params) > 0
-            if isinstance(params, list):
-                return len(params) > 0
-            if isinstance(params, object):
-               return True
-            return False
+                raise JsonRpcException('Invalid "method" field value')                         
 
         def SubValidateErrorObj(errdict):
             '''Checks if Error object has JSON-RPC 2.0 required fields. '''
@@ -173,26 +158,19 @@ def SubParseNotification(jsondict):
             except JsonRpcException as e:                    
                 raise JsonRpcParseError(JsonRpcError.InvalidRequest(str(e)))
             params = jsondict.get('params', None)
-            try:
-                SubValidateParams(params)
-                payload = JsonRpcMessage.Notification(jsondict['method'], params)
-                return JsonRpcParsed(JsonRpcParsedType.NOTIFICATION, payload)
-            except JsonRpcException as e:      
-                raise JsonRpcParseError(JsonRpcError.InvalidParams(str(e)))                    
+            payload = JsonRpcMessage.Notification(jsondict['method'], params)
+            return JsonRpcParsed(JsonRpcParsedType.NOTIFICATION, payload)                   
 
         def SubParseRequest(jsondict):
             '''Parses jsondict, validates JSON-RPC 2.0 Request structure and values.
             Doesn't check JSON-RPC 2.0 "jsonrpc","id", "method".
             Raises `JsonRpcParseError` if parse failed, or params invalid.''' 
             id = jsondict['id'] 
             method = jsondict['method']            
-            params = jsondict.get('params', None) 
-            try:
-                SubValidateParams(params)
-                payload = JsonRpcMessage.Request(id, method, params)
-                return JsonRpcParsed(JsonRpcParsedType.REQUEST, payload)
-            except JsonRpcException as e:
-                raise JsonRpcParseError(JsonRpcError.InvalidParams(str(e)))
+            params = jsondict.get('params', None)             
+            payload = JsonRpcMessage.Request(id, method, params)
+            return JsonRpcParsed(JsonRpcParsedType.REQUEST, payload)
+            
 
         def SubParseSuccessResponse(jsondict):
             '''Parses jsondict, validates JSON-RPC 2.0 Response Success structure and values.

tests/test_jsonrpc.py

@@ -177,7 +177,7 @@ def testParseNotificationReq(self):
         )
         actual = JsonRpcParsed.Parse(testReqJson)
         testutils.assertEqualObjects(expected, actual)
-        
+
     def testParseSuccessRes(self):
         '''Checks if JSON-RPC 2.0 Success object parsed correct'''
         testReqJson = '''