mirror of https://github.com/zulip/zulip.git
481 lines
18 KiB
Python
481 lines
18 KiB
Python
from collections import defaultdict
|
|
from collections.abc import Callable, MutableMapping, Sequence
|
|
from dataclasses import dataclass, field
|
|
from functools import wraps
|
|
from types import FunctionType
|
|
from typing import Any, Concatenate, Generic, Literal, Optional, TypeVar, cast, overload
|
|
|
|
import orjson
|
|
from django.conf import settings
|
|
from django.core.exceptions import ValidationError
|
|
from django.http import HttpRequest, HttpResponse
|
|
from django.utils.translation import gettext as _
|
|
from typing_extensions import ParamSpec, override
|
|
|
|
from zerver.lib import rate_limiter
|
|
from zerver.lib.exceptions import ErrorCode, InvalidJSONError, JsonableError
|
|
from zerver.lib.notes import BaseNotes
|
|
from zerver.lib.response import MutableJsonResponse
|
|
from zerver.lib.types import Validator
|
|
from zerver.lib.validator import check_anything
|
|
from zerver.models import Client, Realm
|
|
|
|
if settings.ZILENCER_ENABLED:
|
|
from zilencer.models import RemoteZulipServer
|
|
|
|
|
|
@dataclass
|
|
class RequestNotes(BaseNotes[HttpRequest, "RequestNotes"]):
|
|
"""This class contains extra metadata that Zulip associated with a
|
|
Django HttpRequest object. See the docstring for BaseNotes for
|
|
details on how it works.
|
|
|
|
Note that most Optional fields will be definitely not None once
|
|
middleware has run. In the future, we may want to express that in
|
|
the types by having different types EarlyRequestNotes and
|
|
post-middleware RequestNotes types, but for now we have a lot
|
|
of `assert request_notes.foo is not None` when accessing them.
|
|
"""
|
|
|
|
client: Client | None = None
|
|
client_name: str | None = None
|
|
client_version: str | None = None
|
|
log_data: MutableMapping[str, Any] | None = None
|
|
rate_limit: str | None = None
|
|
requester_for_logs: str | None = None
|
|
# We use realm_cached to indicate whether the realm is cached or not.
|
|
# Because the default value of realm is None, which can indicate "unset"
|
|
# and "nonexistence" at the same time.
|
|
realm: Realm | None = None
|
|
has_fetched_realm: bool = False
|
|
set_language: str | None = None
|
|
ratelimits_applied: list[rate_limiter.RateLimitResult] = field(default_factory=list)
|
|
query: str | None = None
|
|
error_format: str | None = None
|
|
saved_response: HttpResponse | None = None
|
|
tornado_handler_id: int | None = None
|
|
processed_parameters: set[str] = field(default_factory=set)
|
|
remote_server: Optional["RemoteZulipServer"] = None
|
|
is_webhook_view: bool = False
|
|
|
|
@classmethod
|
|
@override
|
|
def init_notes(cls) -> "RequestNotes":
|
|
return RequestNotes()
|
|
|
|
|
|
class RequestConfusingParamsError(JsonableError):
|
|
code = ErrorCode.REQUEST_CONFUSING_VAR
|
|
data_fields = ["var_name1", "var_name2"]
|
|
|
|
def __init__(self, var_name1: str, var_name2: str) -> None:
|
|
self.var_name1: str = var_name1
|
|
self.var_name2: str = var_name2
|
|
|
|
@staticmethod
|
|
@override
|
|
def msg_format() -> str:
|
|
return _("Can't decide between '{var_name1}' and '{var_name2}' arguments")
|
|
|
|
|
|
class RequestVariableMissingError(JsonableError):
|
|
code = ErrorCode.REQUEST_VARIABLE_MISSING
|
|
data_fields = ["var_name"]
|
|
|
|
def __init__(self, var_name: str) -> None:
|
|
self.var_name: str = var_name
|
|
|
|
@staticmethod
|
|
@override
|
|
def msg_format() -> str:
|
|
return _("Missing '{var_name}' argument")
|
|
|
|
|
|
class RequestVariableConversionError(JsonableError):
|
|
code = ErrorCode.REQUEST_VARIABLE_INVALID
|
|
data_fields = ["var_name", "bad_value"]
|
|
|
|
def __init__(self, var_name: str, bad_value: Any) -> None:
|
|
self.var_name: str = var_name
|
|
self.bad_value = bad_value
|
|
|
|
@staticmethod
|
|
@override
|
|
def msg_format() -> str:
|
|
return _("Bad value for '{var_name}': {bad_value}")
|
|
|
|
|
|
# Used in conjunction with @has_request_variables, below
|
|
ResultT = TypeVar("ResultT")
|
|
|
|
|
|
class _REQ(Generic[ResultT]):
|
|
# NotSpecified is a sentinel value for determining whether a
|
|
# default value was specified for a request variable. We can't
|
|
# use None because that could be a valid, user-specified default
|
|
class _NotSpecified:
|
|
pass
|
|
|
|
NotSpecified = _NotSpecified()
|
|
|
|
def __init__(
|
|
self,
|
|
whence: str | None = None,
|
|
*,
|
|
converter: Callable[[str, str], ResultT] | None = None,
|
|
default: _NotSpecified | ResultT | None = NotSpecified,
|
|
json_validator: Validator[ResultT] | None = None,
|
|
str_validator: Validator[ResultT] | None = None,
|
|
argument_type: str | None = None,
|
|
intentionally_undocumented: bool = False,
|
|
documentation_pending: bool = False,
|
|
aliases: Sequence[str] = [],
|
|
path_only: bool = False,
|
|
) -> None:
|
|
"""whence: the name of the request variable that should be used
|
|
for this parameter. Defaults to a request variable of the
|
|
same name as the parameter.
|
|
|
|
converter: a function that takes a string and returns a new
|
|
value. If specified, this will be called on the request
|
|
variable value before passing to the function
|
|
|
|
default: a value to be used for the argument if the parameter
|
|
is missing in the request
|
|
|
|
json_validator: similar to converter, but takes an already
|
|
parsed JSON data structure. If specified, we will parse the
|
|
JSON request variable value before passing to the function
|
|
|
|
str_validator: Like json_validator, but doesn't parse JSON
|
|
first.
|
|
|
|
argument_type: pass 'body' to extract the parsed JSON
|
|
corresponding to the request body
|
|
|
|
aliases: alternate names for the POST var
|
|
|
|
path_only: Used for parameters included in the URL that we still want
|
|
to validate via REQ's hooks.
|
|
|
|
"""
|
|
|
|
if argument_type == "body" and converter is None and json_validator is None:
|
|
# legacy behavior
|
|
json_validator = cast(Callable[[str, object], ResultT], check_anything)
|
|
|
|
self.post_var_name = whence
|
|
self.func_var_name: str | None = None
|
|
self.converter = converter
|
|
self.json_validator = json_validator
|
|
self.str_validator = str_validator
|
|
self.default = default
|
|
self.argument_type = argument_type
|
|
self.aliases = aliases
|
|
self.intentionally_undocumented = intentionally_undocumented
|
|
self.documentation_pending = documentation_pending
|
|
self.path_only = path_only
|
|
|
|
assert converter is None or (
|
|
json_validator is None and str_validator is None
|
|
), "converter and json_validator are mutually exclusive"
|
|
assert (
|
|
json_validator is None or str_validator is None
|
|
), "json_validator and str_validator are mutually exclusive"
|
|
|
|
|
|
# This factory function ensures that mypy can correctly analyze REQ.
|
|
#
|
|
# Note that REQ claims to return a type matching that of the parameter
|
|
# of which it is the default value, allowing type checking of view
|
|
# functions using has_request_variables. In reality, REQ returns an
|
|
# instance of class _REQ to enable the decorator to scan the parameter
|
|
# list for _REQ objects and patch the parameters as the true types.
|
|
#
|
|
# See also this documentation to learn how @overload helps here.
|
|
# https://zulip.readthedocs.io/en/latest/testing/mypy.html#using-overload-to-accurately-describe-variations
|
|
#
|
|
# Overload 1: converter
|
|
@overload
|
|
def REQ(
|
|
whence: str | None = ...,
|
|
*,
|
|
converter: Callable[[str, str], ResultT],
|
|
default: ResultT = ...,
|
|
argument_type: Literal["body"] | None = ...,
|
|
intentionally_undocumented: bool = ...,
|
|
documentation_pending: bool = ...,
|
|
aliases: Sequence[str] = ...,
|
|
path_only: bool = ...,
|
|
) -> ResultT: ...
|
|
|
|
|
|
# Overload 2: json_validator
|
|
@overload
|
|
def REQ(
|
|
whence: str | None = ...,
|
|
*,
|
|
default: ResultT = ...,
|
|
json_validator: Validator[ResultT],
|
|
argument_type: Literal["body"] | None = ...,
|
|
intentionally_undocumented: bool = ...,
|
|
documentation_pending: bool = ...,
|
|
aliases: Sequence[str] = ...,
|
|
path_only: bool = ...,
|
|
) -> ResultT: ...
|
|
|
|
|
|
# Overload 3: no converter/json_validator, default: str or unspecified, argument_type=None
|
|
@overload
|
|
def REQ(
|
|
whence: str | None = ...,
|
|
*,
|
|
default: str = ...,
|
|
str_validator: Validator[str] | None = ...,
|
|
intentionally_undocumented: bool = ...,
|
|
documentation_pending: bool = ...,
|
|
aliases: Sequence[str] = ...,
|
|
path_only: bool = ...,
|
|
) -> str: ...
|
|
|
|
|
|
# Overload 4: no converter/validator, default=None, argument_type=None
|
|
@overload
|
|
def REQ(
|
|
whence: str | None = ...,
|
|
*,
|
|
default: None,
|
|
str_validator: Validator[str] | None = ...,
|
|
intentionally_undocumented: bool = ...,
|
|
documentation_pending: bool = ...,
|
|
aliases: Sequence[str] = ...,
|
|
path_only: bool = ...,
|
|
) -> str | None: ...
|
|
|
|
|
|
# Overload 5: argument_type="body"
|
|
@overload
|
|
def REQ(
|
|
whence: str | None = ...,
|
|
*,
|
|
default: ResultT = ...,
|
|
argument_type: Literal["body"],
|
|
intentionally_undocumented: bool = ...,
|
|
documentation_pending: bool = ...,
|
|
aliases: Sequence[str] = ...,
|
|
path_only: bool = ...,
|
|
) -> ResultT: ...
|
|
|
|
|
|
# Implementation
|
|
def REQ(
|
|
whence: str | None = None,
|
|
*,
|
|
converter: Callable[[str, str], ResultT] | None = None,
|
|
default: _REQ._NotSpecified | ResultT = _REQ.NotSpecified,
|
|
json_validator: Validator[ResultT] | None = None,
|
|
str_validator: Validator[ResultT] | None = None,
|
|
argument_type: str | None = None,
|
|
intentionally_undocumented: bool = False,
|
|
documentation_pending: bool = False,
|
|
aliases: Sequence[str] = [],
|
|
path_only: bool = False,
|
|
) -> ResultT:
|
|
return cast(
|
|
ResultT,
|
|
_REQ(
|
|
whence,
|
|
converter=converter,
|
|
default=default,
|
|
json_validator=json_validator,
|
|
str_validator=str_validator,
|
|
argument_type=argument_type,
|
|
intentionally_undocumented=intentionally_undocumented,
|
|
documentation_pending=documentation_pending,
|
|
aliases=aliases,
|
|
path_only=path_only,
|
|
),
|
|
)
|
|
|
|
|
|
arguments_map: dict[str, list[str]] = defaultdict(list)
|
|
ParamT = ParamSpec("ParamT")
|
|
ReturnT = TypeVar("ReturnT")
|
|
|
|
|
|
# Extracts variables from the request object and passes them as
|
|
# named function arguments. The request object must be the first
|
|
# argument to the function.
|
|
#
|
|
# To use, assign a function parameter a default value that is an
|
|
# instance of the _REQ class. That parameter will then be automatically
|
|
# populated from the HTTP request. The request object must be the
|
|
# first argument to the decorated function.
|
|
#
|
|
# This should generally be the innermost (syntactically bottommost)
|
|
# decorator applied to a view, since other decorators won't preserve
|
|
# the default parameter values used by has_request_variables.
|
|
#
|
|
# Note that this can't be used in helper functions which are not
|
|
# expected to call json_success or raise JsonableError, as it uses JsonableError
|
|
# internally when it encounters an error
|
|
def has_request_variables(
|
|
req_func: Callable[Concatenate[HttpRequest, ParamT], ReturnT],
|
|
) -> Callable[Concatenate[HttpRequest, ParamT], ReturnT]:
|
|
num_params = req_func.__code__.co_argcount
|
|
default_param_values = cast(FunctionType, req_func).__defaults__
|
|
if default_param_values is None:
|
|
default_param_values = ()
|
|
num_default_params = len(default_param_values)
|
|
default_param_names = req_func.__code__.co_varnames[num_params - num_default_params :]
|
|
|
|
post_params = []
|
|
|
|
view_func_full_name = f"{req_func.__module__}.{req_func.__name__}"
|
|
|
|
for name, value in zip(default_param_names, default_param_values, strict=False):
|
|
if isinstance(value, _REQ):
|
|
value.func_var_name = name
|
|
if value.post_var_name is None:
|
|
value.post_var_name = name
|
|
post_params.append(value)
|
|
|
|
# Record arguments that should be documented so that our
|
|
# automated OpenAPI docs tests can compare these against the code.
|
|
if (
|
|
not value.intentionally_undocumented
|
|
and not value.documentation_pending
|
|
and not value.path_only
|
|
):
|
|
arguments_map[view_func_full_name].append(value.post_var_name)
|
|
|
|
@wraps(req_func)
|
|
def _wrapped_req_func(
|
|
request: HttpRequest, /, *args: ParamT.args, **kwargs: ParamT.kwargs
|
|
) -> ReturnT:
|
|
request_notes = RequestNotes.get_notes(request)
|
|
for param in post_params:
|
|
func_var_name = param.func_var_name
|
|
if param.path_only:
|
|
# For path_only parameters, they should already have
|
|
# been passed via the URL, so there's no need for REQ
|
|
# to do anything.
|
|
#
|
|
# TODO: Either run validators for path_only parameters
|
|
# or don't declare them using REQ.
|
|
|
|
# no coverage because has_request_variables will be removed once
|
|
# all the endpoints have been migrated to use typed_endpoint.
|
|
assert func_var_name in kwargs # nocoverage
|
|
if func_var_name in kwargs:
|
|
continue # nocoverage
|
|
assert func_var_name is not None
|
|
|
|
post_var_name: str | None
|
|
|
|
if param.argument_type == "body":
|
|
post_var_name = "request"
|
|
try:
|
|
val = request.body.decode(request.encoding or "utf-8")
|
|
except UnicodeDecodeError:
|
|
raise JsonableError(_("Malformed payload"))
|
|
else:
|
|
# This is a view bug, not a user error, and thus should throw a 500.
|
|
assert param.argument_type is None, "Invalid argument type"
|
|
|
|
post_var_names = [param.post_var_name]
|
|
post_var_names += param.aliases
|
|
post_var_name = None
|
|
|
|
for req_var in post_var_names:
|
|
assert req_var is not None
|
|
if req_var in request.POST:
|
|
val = request.POST[req_var]
|
|
request_notes.processed_parameters.add(req_var)
|
|
elif req_var in request.GET:
|
|
val = request.GET[req_var]
|
|
request_notes.processed_parameters.add(req_var)
|
|
else:
|
|
# This is covered by test_REQ_aliases, but coverage.py
|
|
# fails to recognize this for some reason.
|
|
continue # nocoverage
|
|
if post_var_name is not None:
|
|
raise RequestConfusingParamsError(post_var_name, req_var)
|
|
post_var_name = req_var
|
|
|
|
if post_var_name is None:
|
|
post_var_name = param.post_var_name
|
|
assert post_var_name is not None
|
|
if param.default is _REQ.NotSpecified:
|
|
raise RequestVariableMissingError(post_var_name)
|
|
kwargs[func_var_name] = param.default
|
|
continue
|
|
|
|
if param.converter is not None:
|
|
try:
|
|
val = param.converter(post_var_name, val)
|
|
except JsonableError:
|
|
raise
|
|
except Exception:
|
|
raise RequestVariableConversionError(post_var_name, val)
|
|
|
|
# json_validator is like converter, but doesn't handle JSON parsing; we do.
|
|
if param.json_validator is not None:
|
|
try:
|
|
val = orjson.loads(val)
|
|
except orjson.JSONDecodeError:
|
|
if param.argument_type == "body":
|
|
raise InvalidJSONError(_("Malformed JSON"))
|
|
raise JsonableError(
|
|
_('Argument "{name}" is not valid JSON.').format(name=post_var_name)
|
|
)
|
|
|
|
try:
|
|
val = param.json_validator(post_var_name, val)
|
|
except ValidationError as error:
|
|
raise JsonableError(error.message)
|
|
|
|
# str_validators is like json_validator, but for direct strings (no JSON parsing).
|
|
if param.str_validator is not None:
|
|
try:
|
|
val = param.str_validator(post_var_name, val)
|
|
except ValidationError as error:
|
|
raise JsonableError(error.message)
|
|
|
|
kwargs[func_var_name] = val
|
|
|
|
return_value = req_func(request, *args, **kwargs)
|
|
|
|
if (
|
|
isinstance(return_value, MutableJsonResponse)
|
|
and not request_notes.is_webhook_view
|
|
# Implemented only for 200 responses.
|
|
# TODO: Implement returning unsupported ignored parameters for 400
|
|
# JSON error responses. This is complex because has_request_variables
|
|
# can be called multiple times, so when an error response is raised,
|
|
# there may be supported parameters that have not yet been processed,
|
|
# which could lead to inaccurate output.
|
|
and 200 <= return_value.status_code < 300
|
|
):
|
|
ignored_parameters = {*request.POST, *request.GET}.difference(
|
|
request_notes.processed_parameters
|
|
)
|
|
|
|
# This will be called each time a function decorated with
|
|
# has_request_variables returns a MutableJsonResponse with a
|
|
# success status_code. Because a shared processed_parameters
|
|
# value is checked each time, the value for the
|
|
# ignored_parameters_unsupported key is either added/updated
|
|
# to the response data or it is removed in the case that all
|
|
# of the request parameters have been processed.
|
|
if ignored_parameters:
|
|
return_value.get_data()["ignored_parameters_unsupported"] = sorted(
|
|
ignored_parameters
|
|
)
|
|
else:
|
|
return_value.get_data().pop("ignored_parameters_unsupported", None)
|
|
|
|
return return_value
|
|
|
|
return _wrapped_req_func
|