Django中间件返回HttpResponseRedirect后请求未终止问题排查

看起来你遇到的问题是中间件返回HttpResponseRedirect时，请求没有像返回HttpResponse那样终止，反而继续执行并重新加载profile页面。结合你的代码和Django 2.1.7的特性，我来帮你分析并解决这个问题：

可能的原因 #

1. MiddlewareMixin的适配问题：虽然MiddlewareMixin是为了兼容旧版中间件写法，但在某些场景下，它对process_request返回值的处理可能不如新版中间件写法直接，导致请求没有被及时终止。
2. 缺少用户登录状态判断：你的中间件没有判断用户是否已登录，匿名用户的request.user是AnonymousUser，它的first_name为空字符串，会触发不必要的重定向逻辑，干扰正常流程。
3. 资料完整性判断不够严谨：Django的User模型中first_name和last_name默认值是None而非空字符串，你的判断条件request.user.first_name == ""会漏掉这种情况。

解决方案 #

1. 改用Django 1.10+推荐的中间件写法 #

放弃MiddlewareMixin，使用新的中间件格式（通过__init__和__call__方法实现），这样能更直接地控制请求流程：

from django.urls import reverse
from django.http import HttpResponseRedirect
import pytz
from django.utils import timezone

class TimezoneMiddleware:
    def __init__(self, get_response):
        self.get_response = get_response

    def __call__(self, request):
        request.session['timezone'] = 'Asia/Kolkata'
        timezone.activate(pytz.timezone('Asia/Kolkata'))
        response = self.get_response(request)
        return response

class CheckProfileMiddleware:
    def __init__(self, get_response):
        self.get_response = get_response

    def __call__(self, request):
        # 先判断用户是否已登录，避免匿名用户触发逻辑
        if request.user.is_authenticated:
            # 更严谨的资料完整性判断（兼容None和空字符串）
            if not request.user.first_name or not request.user.last_name:
                if 'profile' not in request.path_info:
                    request.session['is_incomplete'] = 1
                    # 直接返回重定向，不执行后续view和中间件
                    return HttpResponseRedirect(reverse('profile'))
            else:
                request.session['is_incomplete'] = 0
        
        # 继续执行后续请求流程
        response = self.get_response(request)
        return response

2. 确认中间件顺序 #

确保CheckProfileMiddleware放在AuthenticationMiddleware之后（你的配置已经符合），这样能保证request.user已经被正确初始化：

MIDDLEWARE = [
    'django.middleware.security.SecurityMiddleware',
    'django.contrib.sessions.middleware.SessionMiddleware',
    'django.middleware.common.CommonMiddleware',
    'django.middleware.csrf.CsrfViewMiddleware',
    'django.contrib.auth.middleware.AuthenticationMiddleware',
    'users.middleware.TimezoneMiddleware',
    'django.contrib.messages.middleware.MessageMiddleware',
    'django.middleware.clickjacking.XFrameOptionsMiddleware',
    'users.middleware.CheckProfileMiddleware',  # 保持在AuthenticationMiddleware之后
]

3. 优化视图中的冗余逻辑 #

你的profile视图可以简化一些查询逻辑，同时确保资料更新后不会触发重复重定向：

@login_required
def profile(request):
    user = request.user  # 直接使用request.user，无需重复查询
    try:
        user_profile = Profile.objects.get(user=user)
    except Profile.DoesNotExist:
        # 处理Profile不存在的边界情况，创建默认实例
        user_profile = Profile.objects.create(user=user)
    
    if request.method == 'POST':
        u_form = UserUpdateForm(request.POST, instance=user)
        p_form = ProfileUpdateForm(request.POST, request.FILES, instance=user_profile)
        if u_form.is_valid() and p_form.is_valid():
            u_form.save()
            p_form.save()
            messages.success(request, f'Your Profile has been updated!')
            return redirect('profile')
    else:
        u_form = UserUpdateForm(instance=user)
        p_form = ProfileUpdateForm(instance=user_profile)
    
    context = {
        'u_form': u_form,
        'p_form': p_form,
        'users': user,
    }
    return render(request, 'profile.html', context)

为什么这样能解决问题？ #

新版中间件的__call__方法中，当你直接返回HttpResponseRedirect时，不会调用self.get_response(request)，这就直接终止了后续的视图和中间件执行流程，完全符合你想要的“请求终止并重定向”的效果。同时，添加登录状态判断和更严谨的字段检查，能避免不必要的逻辑触发，让流程更清晰。

内容的提问来源于stack exchange，提问作者Subham