-
-
Notifications
You must be signed in to change notification settings - Fork 125
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
Updated Security Section #242
Conversation
Created draft version with details of Cantina's review
Added the details about the security bug made by Certora and refactored the text
Ready for your review @PaulRBerg |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @0xJayPi, thanks for your PR. I have reviewed it now and I want to make some changes, but it looks like I am not allowed to do that:
! [rejected] main -> main (non-fast-forward)
Could you please give me permission to push to your branch?
Alternatively, please make your changes on a new branch (not on main
). That will make it work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The standard place to place security-related information is the SECURITY.md
file. Let's move this file over there.
Apologies, first time I don't create a branch for simplicity. I'll make the needed changes next week (on vacations leave right now) |
@PaulRBerg. I just sent you an invite to my repo. If still needed, I can just move forward and do a new branch and move everything there. Please let me know |
@0xJayPi I would prefer not to be added as a collaborator to your repo. Please just create a new PR with a different branch. Thank you. |
Description
Added reference to security reviews of the codebase
Changes
README.md
audits/AUDITS.md
Related Issue
Related to #197