Skip to content

Hardening suggestions for django-twitter / 11-comments-model-api-tests#19

Open
pixeebot[bot] wants to merge 2 commits into
11-comments-model-api-testsfrom
pixeebot/11-comments-model-api-tests
Open

Hardening suggestions for django-twitter / 11-comments-model-api-tests#19
pixeebot[bot] wants to merge 2 commits into
11-comments-model-api-testsfrom
pixeebot/11-comments-model-api-tests

Conversation

@pixeebot

@pixeebot pixeebot Bot commented May 30, 2024

Copy link
Copy Markdown

I've reviewed the recently opened PR (18 - comments api model and tests) and have identified some area(s) that could benefit from additional hardening measures.

These changes should help prevent potential security vulnerabilities and improve overall code quality.

Thank you for your consideration!
🧚🤖 Powered by Pixeebot

💬Feedback | 👥Community | 📚Docs

@pixeebot pixeebot Bot requested a review from taoerman May 30, 2024 23:27
Comment thread twitter/settings.py

# SECURITY WARNING: don't run with debug turned on in production!
DEBUG = True
DEBUG = False

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Flip Django debug flag to off.

Comment thread twitter/settings.py
# https://docs.djangoproject.com/en/3.1/howto/static-files/

STATIC_URL = '/static/'
SESSION_COOKIE_SECURE = True

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Sets Django's SESSION_COOKIE_SECURE flag if off or missing.

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.

0 participants