diff options
| author | Aymeric Augustin <aymeric.augustin@m4x.org> | 2022-05-22 08:26:21 +0200 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2022-05-25 10:58:48 +0200 |
| commit | 8cf4de206ce4e1fe1e06a55a30c4e6bebe7cf10a (patch) | |
| tree | f9abbdabd7e7da2f237092458b51d4407607e86c /django | |
| parent | aff649a3bd0c4fd8f7d859464f0eb907e877443f (diff) | |
Normalized decorator style for functools.wraps.
Diffstat (limited to 'django')
| -rw-r--r-- | django/views/decorators/clickjacking.py | 9 | ||||
| -rw-r--r-- | django/views/decorators/common.py | 3 | ||||
| -rw-r--r-- | django/views/decorators/csrf.py | 3 |
3 files changed, 10 insertions, 5 deletions
diff --git a/django/views/decorators/clickjacking.py b/django/views/decorators/clickjacking.py index 005d942c76..8fa49ddb80 100644 --- a/django/views/decorators/clickjacking.py +++ b/django/views/decorators/clickjacking.py @@ -12,13 +12,14 @@ def xframe_options_deny(view_func): ... """ + @wraps(view_func) def wrapper_view(*args, **kwargs): resp = view_func(*args, **kwargs) if resp.get("X-Frame-Options") is None: resp["X-Frame-Options"] = "DENY" return resp - return wraps(view_func)(wrapper_view) + return wrapper_view def xframe_options_sameorigin(view_func): @@ -32,13 +33,14 @@ def xframe_options_sameorigin(view_func): ... """ + @wraps(view_func) def wrapper_view(*args, **kwargs): resp = view_func(*args, **kwargs) if resp.get("X-Frame-Options") is None: resp["X-Frame-Options"] = "SAMEORIGIN" return resp - return wraps(view_func)(wrapper_view) + return wrapper_view def xframe_options_exempt(view_func): @@ -51,9 +53,10 @@ def xframe_options_exempt(view_func): ... """ + @wraps(view_func) def wrapper_view(*args, **kwargs): resp = view_func(*args, **kwargs) resp.xframe_options_exempt = True return resp - return wraps(view_func)(wrapper_view) + return wrapper_view diff --git a/django/views/decorators/common.py b/django/views/decorators/common.py index a973389549..fe4d58f591 100644 --- a/django/views/decorators/common.py +++ b/django/views/decorators/common.py @@ -8,8 +8,9 @@ def no_append_slash(view_func): """ # view_func.should_append_slash = False would also work, but decorators are # nicer if they don't have side effects, so return a new function. + @wraps(view_func) def wrapper_view(*args, **kwargs): return view_func(*args, **kwargs) wrapper_view.should_append_slash = False - return wraps(view_func)(wrapper_view) + return wrapper_view diff --git a/django/views/decorators/csrf.py b/django/views/decorators/csrf.py index e6c9cb7660..40ac58bc1d 100644 --- a/django/views/decorators/csrf.py +++ b/django/views/decorators/csrf.py @@ -50,8 +50,9 @@ def csrf_exempt(view_func): """Mark a view function as being exempt from the CSRF view protection.""" # view_func.csrf_exempt = True would also work, but decorators are nicer # if they don't have side effects, so return a new function. + @wraps(view_func) def wrapper_view(*args, **kwargs): return view_func(*args, **kwargs) wrapper_view.csrf_exempt = True - return wraps(view_func)(wrapper_view) + return wrapper_view |