Skip to content

Comments

Add article management dashboard and enhance security features#19

Merged
dendencat merged 1 commit intodevelopfrom
chore/merge-main-into-develop
Sep 16, 2025
Merged

Add article management dashboard and enhance security features#19
dendencat merged 1 commit intodevelopfrom
chore/merge-main-into-develop

Conversation

@dendencat
Copy link
Owner

Introduce a sidebar component and a dashboard template for managing articles, including pagination and delete functionality. Implement user authentication and improve security by enforcing CSRF protection and sanitizing markdown content. Update the project structure for better modularity and add end-to-end tests for critical features. Adjust settings for production readiness and enhance the overall user interface.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

Comment on lines 10 to 12
<form method="post">
{% if not IS_TESTING %}
{% csrf_token %}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P1] Skipping CSRF token when TESTING=True breaks login flows

When the TESTING flag is set (either via environment variable or the pytest detection in settings.py), the login template stops rendering {% csrf_token %} but the view continues to be protected by CsrfViewMiddleware. Any browser-based login against a server started with TESTING=True will therefore submit no CSRF token and receive a 403 before the view executes, effectively disabling sign‑in for staging/test environments. The tests still pass because Django’s test client doesn’t enforce CSRF, so this regression is silent.

Useful? React with 👍 / 👎.

@dendencat dendencat merged commit 5566c7a into develop Sep 16, 2025
3 checks passed
@dendencat dendencat deleted the chore/merge-main-into-develop branch September 16, 2025 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant