Raise for index selector out of range.

Author: jg-rp

CHANGELOG.md

@@ -19,6 +19,7 @@
 - Removed support for dotted index selectors.
 - Raise a `JSONPathSyntaxError` for unescaped whitespace and control characters.
 - Raise a `JSONPathSyntaxError` for empty selector segments.
+- Raise a `JSONPathIndexError` if an index selector is out of range.
 
 ## Version 0.4.0
 

jsonpath/env.py

@@ -85,6 +85,9 @@ class attributes `root_token`, `self_token` and `filter_context_token`.
     union_token: str = "|"
     filter_context_token: str = "#"
 
+    max_int_index = (2**53) - 1
+    min_int_index = -(2**53) + 1
+
     # Override these to customize path tokenization and parsing.
     lexer_class: Type[Lexer] = Lexer
     parser_class: Type[Parser] = Parser

jsonpath/exceptions.py

@@ -45,6 +45,10 @@ class JSONPathTypeError(JSONPathError):
     """
 
 
+class JSONPathIndexError(JSONPathError):
+    """An exception raised when an array index is out of range."""
+
+
 class JSONPathNameError(JSONPathError):
     """An exception raised when an unknown function extension is called."""
 

jsonpath/selectors.py

@@ -15,6 +15,7 @@
 from typing import TypeVar
 from typing import Union
 
+from .exceptions import JSONPathIndexError
 from .exceptions import JSONPathTypeError
 from .match import JSONPathMatch
 
@@ -113,6 +114,9 @@ def __init__(
         token: Token,
         index: int,
     ) -> None:
+        if index < env.min_int_index or index > env.max_int_index:
+            raise JSONPathIndexError("index out of range", token=token)
+
         super().__init__(env=env, token=token)
         self.index = index
         self._as_key = str(self.index)
@@ -200,6 +204,7 @@ def __init__(
         stop: Optional[int] = None,
         step: Optional[int] = None,
     ) -> None:
+        # TODO: raise if start, stop or step are out of range
         super().__init__(env=env, token=token)
         self.slice = slice(start, stop, step)
 

tests/compliance.py

@@ -87,3 +87,6 @@ def test_invalid_selectors(case: Case) -> None:
 
     with pytest.raises(jsonpath.JSONPathError):
         jsonpath.compile(case.selector)
+
+
+# TODO: async invalid cases