Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'Security' content added, updated icon links/html, and general formatting changes #175

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

lbenno
Copy link

@lbenno lbenno commented Apr 21, 2024

After completing the 'Security' Extension tutorial it is still possible to view a draft post (draft_post_detail.html) without being logged in if the User navigates the to post via the address bar, because the post_detail() method is not protected by the @login_required decorator. I think it's important for people to understand that just because you can't navigate from the home page, the url itself may still be vulnerable to unauthorised access. So, I've added a section at the end of the security Extension to include validation for published vs draft posts.

At some point in the past icons used in the main tutorial were changed from glyphicons to .svg files downloaded from bootstrap, but seems this had not been reflected in the corresponding sections of the Extensions tutorial. So, I've updated references to that code from the main tutorial, and replaced the new icon instructions included in the Extension tutorial.

Updated code block formatting to be more consistent with main tutorial where the file name is listed directly above the code block.

Hope this is helpful :-)

Lauren Benson
Berlin, Germany

PS: this is my first ever pull request so I hope I did it ok :-)

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