INCR-146 [Python2 > Python3] (#20488)
* INCR-146: Run python-modernize on openedx/core/djangoapps/cors_csrf * INCR-146: Disable pylint warning for import-error
This commit is contained in:
@@ -2,6 +2,8 @@
|
||||
Manage cross-domain configuration.
|
||||
"""
|
||||
|
||||
from __future__ import absolute_import
|
||||
|
||||
from config_models.admin import ConfigurationModelAdmin
|
||||
from django.contrib import admin
|
||||
|
||||
|
||||
@@ -1,6 +1,8 @@
|
||||
"""
|
||||
Django Rest Framework Authentication classes for cross-domain end-points.
|
||||
"""
|
||||
from __future__ import absolute_import
|
||||
|
||||
from django.middleware.csrf import CsrfViewMiddleware
|
||||
from rest_framework import authentication
|
||||
|
||||
|
||||
@@ -1,4 +1,6 @@
|
||||
"""Decorators for cross-domain CSRF. """
|
||||
from __future__ import absolute_import
|
||||
|
||||
from django.views.decorators.csrf import ensure_csrf_cookie
|
||||
|
||||
|
||||
|
||||
@@ -1,7 +1,9 @@
|
||||
"""Helper methods for CORS and CSRF checks. """
|
||||
from __future__ import absolute_import
|
||||
|
||||
import contextlib
|
||||
import logging
|
||||
import urlparse
|
||||
import six.moves.urllib.parse # pylint: disable=import-error
|
||||
|
||||
from django.conf import settings
|
||||
|
||||
@@ -24,7 +26,7 @@ def is_cross_domain_request_allowed(request):
|
||||
|
||||
"""
|
||||
referer = request.META.get('HTTP_REFERER')
|
||||
referer_parts = urlparse.urlparse(referer) if referer else None
|
||||
referer_parts = six.moves.urllib.parse.urlparse(referer) if referer else None
|
||||
referer_hostname = referer_parts.hostname if referer_parts is not None else None
|
||||
|
||||
# Use CORS_ALLOW_INSECURE *only* for development and testing environments;
|
||||
|
||||
@@ -42,6 +42,8 @@ CSRF cookie.
|
||||
|
||||
"""
|
||||
|
||||
from __future__ import absolute_import
|
||||
|
||||
import logging
|
||||
|
||||
from django.conf import settings
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
# -*- coding: utf-8 -*-
|
||||
from __future__ import unicode_literals
|
||||
from __future__ import absolute_import, unicode_literals
|
||||
|
||||
from django.db import migrations, models
|
||||
import django.db.models.deletion
|
||||
|
||||
@@ -1,4 +1,6 @@
|
||||
"""Models for cross-domain configuration. """
|
||||
from __future__ import absolute_import
|
||||
|
||||
from config_models.models import ConfigurationModel
|
||||
from django.db import models
|
||||
from django.utils.translation import ugettext_lazy as _
|
||||
|
||||
@@ -1,4 +1,6 @@
|
||||
"""Tests for the CORS CSRF version of Django Rest Framework's SessionAuthentication."""
|
||||
from __future__ import absolute_import
|
||||
|
||||
from mock import patch
|
||||
|
||||
from django.middleware.csrf import get_token
|
||||
|
||||
@@ -1,4 +1,6 @@
|
||||
"""Tests for cross-domain CSRF decorators. """
|
||||
from __future__ import absolute_import
|
||||
|
||||
import json
|
||||
import mock
|
||||
from django.http import HttpResponse
|
||||
|
||||
@@ -2,6 +2,8 @@
|
||||
Tests for the CORS CSRF middleware
|
||||
"""
|
||||
|
||||
from __future__ import absolute_import
|
||||
|
||||
from mock import patch, Mock
|
||||
import ddt
|
||||
|
||||
|
||||
@@ -1,5 +1,7 @@
|
||||
"""Tests for cross-domain request views. """
|
||||
|
||||
from __future__ import absolute_import
|
||||
|
||||
import json
|
||||
import unittest
|
||||
|
||||
|
||||
@@ -1,4 +1,6 @@
|
||||
"""Views for enabling cross-domain requests. """
|
||||
from __future__ import absolute_import
|
||||
|
||||
import json
|
||||
import logging
|
||||
|
||||
|
||||
Reference in New Issue
Block a user