Skip to content

Commit c1728bf

Browse files
feat: add login_required to enable generate_description
1 parent 0759970 commit c1728bf

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

news/views.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
from django.db import IntegrityError
88
from django.contrib import messages
99
from django.contrib.auth import get_user_model
10+
from django.contrib.auth.decorators import login_required
1011
from django.contrib.auth.mixins import LoginRequiredMixin, UserPassesTestMixin
1112
from django.contrib.humanize.templatetags import humanize
1213
from django.contrib.messages.views import SuccessMessageMixin
@@ -590,6 +591,7 @@ def post(self, request, *args, **kwargs):
590591
return self.render_to_response(context)
591592

592593

594+
@login_required
593595
@require_POST
594596
def generate_description(request):
595597
"""Generate an AI description from submitted content (synchronous).
@@ -598,8 +600,8 @@ def generate_description(request):
598600
Runs the summarization model inline and returns the result as JSON so the
599601
browser can drop it into the Description field.
600602
601-
NOTE: intentionally not login-gated yet (local testing). This endpoint calls
602-
a paid LLM, so add @login_required (and rate limiting) before it ships.
603+
Login-gated since it calls a paid LLM. NOTE: still no rate limiting — add
604+
per-user throttling before relying on auth alone to bound spend.
603605
"""
604606
title = request.POST.get("title", "").strip()
605607
content = request.POST.get("content", "").strip()

0 commit comments

Comments
 (0)