Skip to content

Conversation

@kikofernandez
Copy link
Contributor

@kikofernandez kikofernandez commented Oct 7, 2024

This PR updates how users should report security vulnerabilities in Erlang/OTP.

I have added a deprecation note on the old post from 2017, remove references to emailing us security vulnerabilities (usually users do not encrypt these messages), and copy-pasted news/111 into news/172 removing also references to bugs.erlang.org (which points to https://github.com/erlang/otp/issues).

I am going to ask @u3s and @IngelaAndin to review this description as security experts in Erlang/OTP :)

@netlify
Copy link

netlify bot commented Oct 7, 2024

Deploy Preview for erlang-org ready!

Name Link
🔨 Latest commit f260669
🔍 Latest deploy log https://app.netlify.com/sites/erlang-org/deploys/6728f0c0cf20ff0008650020
😎 Deploy Preview https://deploy-preview-155--erlang-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@kikofernandez kikofernandez force-pushed the master branch 2 times, most recently from ac185d4 to 8be4e10 Compare October 7, 2024 13:18
@kikofernandez kikofernandez force-pushed the master branch 2 times, most recently from 952bae3 to df07665 Compare October 7, 2024 14:59
@kikofernandez
Copy link
Contributor Author

I am not sure I understand what netlify is complaining about...
I have run make test locally and all seems to work, and I have run make serve and it looks ok.

Copy link
Contributor

@u3s u3s left a comment

Choose a reason for hiding this comment

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

I don't know how this security tab on github really works.
I think, I don't have enough privileges for handling the items properly.

There is some procedure behind handling those items on github, I would like to understand it before we change way of working with community.

@garazdawi
Copy link
Contributor

I pushed a fix to master for the redirect rules. If you rebase the PR, CI should now pass.

@garazdawi
Copy link
Contributor

It is quite simple to add a new page with the security information instead of a news item. We could then link from it from the downloads page?

@kikofernandez
Copy link
Contributor Author

Ok, I have added a new Security page in the header. But I can put some description in the Downloads and add it there.
I thought that the Security possibly deserves its own space to be visible everywhere to everyone.

Please, let me know if anyone disagrees, and propose change :)

@kikofernandez
Copy link
Contributor Author

also some other remaining part regarding news/172. I think I have finished that now

Co-authored-by: Lukas Backström (FKA Larsson) <[email protected]>
@kikofernandez
Copy link
Contributor Author

@IngelaAndin and/or @u3s , I know there were concerns about using Github Securities, at least until Security Masters feel ok using it. Whenever that happens, let me know and I merge the PR :)

This RFC is recommended by govs around the world, CISA, IESG, and has
been accepted by IETF.

security.txt is an accepted standard for website security information
that allows security researchers to report security vulnerabilities
easily.
@kikofernandez
Copy link
Contributor Author

Given that we are discussing this PR soon, I also added a new file to indicate researchers how to tell us about vulnerabilities.
https://securitytxt.org/

This has been recommended by CISA, Nederland Gov, UK Gov, many Swedish websites (even H&M has one), Google, Facebook, Github, etc. It is something small that seems to serve a good purpose. I think we should add it, even if it means placing a ticket to as a reminder to renew the date.

@kikofernandez
Copy link
Contributor Author

Merging on Monday, 14:00 unless someone objects

Co-authored-by: Lukas Backström (FKA Larsson) <[email protected]>
@kikofernandez kikofernandez merged commit ee2a81e into erlang:master Nov 6, 2024
5 checks passed
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.

5 participants