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

Create SECURITY.md #8326

Closed
wants to merge 1 commit into from
Closed

Create SECURITY.md #8326

wants to merge 1 commit into from

Conversation

cldrn
Copy link

@cldrn cldrn commented Dec 26, 2021

A nice place to find the contact info of the team handling security vulnerabilities.

@cldrn
Copy link
Author

cldrn commented Dec 26, 2021

Who should i contact to report a security vulnerability in private?

@qqmyers
Copy link
Member

qqmyers commented Dec 27, 2021

@cldrn - Thanks for the PR - a good idea to make this clearer. If you have an issue to report, security@dataverse.org would be a good contact point.

Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

Not a bad idea but how common is SECURITY.md? GitHub doesn't seem to recommend it:

Screen Shot 2022-01-03 at 10 33 39 AM

For more on GitHub's recommendations: https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/about-community-profiles-for-public-repositories

This issue is related: "Securing Your Installation" section of Installation Guide could cover ongoing security, advisories, private discussion #3215

| 5.1.x | :white_check_mark: |
| 5.0.x | :x: |
| 4.0.x | :white_check_mark: |
| < 4.0 | :x: |
Copy link
Member

Choose a reason for hiding this comment

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

To make this table more clear for the purpose of my comments below, here's a screenshot:

Screen Shot 2022-01-03 at 10 24 35 AM

I don't know if we write this down anywhere but generally we encourage installations to upgrade in order to receive security updates. If a flaw is found in an old version, we're not very likely to put out a point release to fix it.

Tell them where to go, how often they can expect to get an update on a
reported vulnerability, what to expect if the vulnerability is accepted or
declined, etc.
WHERE TO REPORT SECURITY VULNERABILITIES
Copy link
Member

Choose a reason for hiding this comment

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

As @qqmyers already mentioned, security@dataverse.org is the place to report security vulnerabilities. This is mentioned under "Reporting Issues and Contributing" at https://guides.dataverse.org/en/5.8/ as well as CONTRIBUTING.md, the issue template, and https://dataverse.org/contact

This issue is related: "Securing Your Installation" section of Installation Guide could cover ongoing security, advisories, private discussion #3215

@scolapasta scolapasta self-assigned this Jan 3, 2022
@cldrn
Copy link
Author

cldrn commented Jan 3, 2022

Hi,
It is suggested on the tab Security between de Wiki and Insights. GH is pushing for projects to use this as it makes it easier to track vulnerabilities and for security researchers to find the right contact. In there you could also explain the expected response time, procedures in general, etc. For example, I contacted security@dataverse.org and they haven't acknowledged the message so I am not sure if that contact was correct at the time.

@pdurbin
Copy link
Member

pdurbin commented Jan 3, 2022

@cldrn ah, yes, I see where GitHub suggests SECURITY.md now. Thanks. It looks like you've pretty much used the template.

Also, it looks like your ticket numbers are RT#313094 and RT#313114 but we're just starting to get back from winter break. It'll take some time for a reply. You should have at least received automated emails with the ticket numbers above.

As I mentioned at standup this morning, it probably make sense to have someone from the core team decide what to write in SECURITY.md so maybe one of us should take it from here.

@scolapasta scolapasta assigned djbrooke and unassigned scolapasta Jan 3, 2022
@djbrooke
Copy link
Contributor

Hi @cldrn - thanks for this PR. I'm going to close this out and we'll prioritize #3215 to add security.md and to update the Guides with additional security-related information.

@djbrooke djbrooke closed this Jan 24, 2022
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