fix: restrict acess for non admin profile - #203
Conversation
|
@skmbaye There are too many files modified that shouldn't be there. |
There was a problem hiding this comment.
There are so many random pictures please check this and remove all added by you in this folder.
There was a problem hiding this comment.
Ok I'll check this as soon as possible
|
Hello @skmbaye As Eugene mentioned earlier, there are just too many random files and images, so unfortunately I can't review this PR. As a guess for so many changes is probably a file mod change (if this is the case check this out to help you https://www.geeksforgeeks.org/git/how-to-make-git-ignore-file-permission-chmod-changes/) and if not then you are using CRLF instead of LF as your line ending (in this case refer to this nice read https://www.aleksandrhovhannisyan.com/blog/crlf-vs-lf-normalizing-line-endings-in-git/) If both of them doesn't fix the issue then let me know, but I am gussing these should be the core problems. Until you fix these, I can't review the PR. |
a04beba to
1b60355
Compare
|
@gauranshkumar my apologies for the previous mess with line endings. I've cleaned up the branch:
The diff is now 4 files / ~140 lines, all directly related to the IDOR fix and its Dockerfile prerequisite. Ready for review :)! |
No description provided.