You’re not receiving notifications from this thread.
Nested Comment Threads in Rails - Part 2 Discussion
There's a security vulnerability in the code which allows any authenticated user to delete anyone else's comment. While there's a check in view (
if comment.user == current_user) for showing the deletion link, there's no such check when actually destroying the comment.
Relevant code from
def destroy @comment = @commentable.comments.find(params[:id]) @comment.destroy redirect_to @commentable end
There are a number of different ways to fix the security vulnerability. Here's one example:
def destroy @comment = @commentable.comments.where(user: current_user).find(params[:id]) @comment.destroy redirect_to @commentable end
If you're planning to also add editing/updating of comments, you might want to turn this into a reusable helper method like so:
def destroy user_comment.destroy redirect_to @commentable end private def user_comment @user_comment =|| @commentable.comments.where(user: current_user).find(params[:id]) end
The same thing also be done using an authorisation gem like Pundit, if anyone already has that integrated into their app.