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

Proposal for SECURITY.md #231

Merged
merged 15 commits into from
Oct 14, 2022
Merged

Proposal for SECURITY.md #231

merged 15 commits into from
Oct 14, 2022

Conversation

hexsecs
Copy link
Member

@hexsecs hexsecs commented Jan 4, 2022

I took a crack at creating a SECURITY.md file. Fixes #227

First draft
Defined acknowledgement time.
SECURITY.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@pattivacek pattivacek left a comment

Choose a reason for hiding this comment

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

The general idea seems fine. I generally prefer being as open as possible about vulnerabilities, but it makes sense to ask for some time to find solutions or mitigations first. However, this is just a standard, not an implementation, so even if we provide a solution to a newly announced vulnerability, that says nothing about whether a given implementation is actually vulnerable, nor whether our proposed solution is viable for it.

SECURITY.md Outdated Show resolved Hide resolved
SECURITY.md Outdated Show resolved Hide resolved
@tkfu
Copy link
Member

tkfu commented Mar 15, 2022

I think there's one really challenging thing going on here: Uptane has a number of implementations, but most of them are private and proprietary. So while it's useful to have a responsible disclosure process for potentially-exploitable issues in the standard itself, the people who need to act on it are the authors/maintainers of those various implementations.

The problem with that is the automotive industry's proclivity for secrecy. We just don't have a comprehensive list of all the implementations out there (or at least I don't). If the issue has the potential to be exploited in the wild, a good responsible disclosure policy is a lot more complicated than it would be for a typical software project. For example, a serious enough issue could necessitate a recall, and regulatory issues surrounding recalls are complicated and different in different jurisdictions around the world.

@jhdalek55 , I think maybe we ought to put this on the agenda for the next standard meeting: what should a responsible-disclosure policy and process for Uptane look like?

@jhdalek55
Copy link
Contributor

@tkfu I made a note and will include this as an agenda item for our next meeting.

@jhdalek55
Copy link
Contributor

This PR provoked quite a bit of discussion at the 3/29 meeting. A side issue that emerged is that we continue to refer to the Reference Implementation even though this was archived sometime ago and has not been updated for years. PR #47 on the website repository is a first attempt at removing these references.

@jhdalek55
Copy link
Contributor

My notes from the last Uptane meeting do not make it clear if we reached a decision on this. Do we wish to stipulate anything about responsible disclosure on the website? Or is this an issue better addressed as a PURE since the central issue of how we identify, notify, and/or correct problems is such a complicated and potentially serious one?

@iramcdonald
Copy link

iramcdonald commented Apr 11, 2022 via email

@jhdalek55
Copy link
Contributor

Perhaps we can discuss the proposed solution from @iramcdonald above at our next meeting?

@jhdalek55
Copy link
Contributor

I am proposing we move this issue to a PURE. As @iramcdonald noted, there could be consequences if we frame this incorrectly in the Standard or even the Deployment pages.

@trishankatdatadog
Copy link
Member

Should we then close this issue, and open a PURE instead?

@jhdalek55
Copy link
Contributor

I think it would make sense to move this to the PURES repo. @hexsecs any objection to doing so?

@hexsecs
Copy link
Member Author

hexsecs commented Aug 10, 2022

I think it would make sense to move this to the PURES repo. @hexsecs any objection to doing so?

No objection to moving it to a PURE

@jhdalek55
Copy link
Contributor

Thanks. I'll take care of moving it over.

@jhdalek55
Copy link
Contributor

I have opened a pull request for PURE 3 (uptane/pures#8) that converts this request into the PURE format. It is incomplete and requires additional text as well as suggested revisions. @hexsecs @pattivacek and others, please give us your input.

SECURITY.md Outdated Show resolved Hide resolved
Revised to use "errata" instead of "vulnerability"
added 2.0.0
SECURITY.md Outdated Show resolved Hide resolved
SECURITY.md Outdated Show resolved Hide resolved
SECURITY.md Outdated Show resolved Hide resolved
SECURITY.md Outdated Show resolved Hide resolved
SECURITY.md Outdated Show resolved Hide resolved
SECURITY.md Outdated Show resolved Hide resolved
SECURITY.md Outdated Show resolved Hide resolved
SECURITY.md Outdated Show resolved Hide resolved
hexsecs and others added 4 commits September 22, 2022 09:58
I updated document incorporating the great feedback from @jhdalek55

Co-authored-by: Lois Anne DeLong <[email protected]>
Fixing some grammar issues.
SECURITY.md Outdated Show resolved Hide resolved
mnm678
mnm678 previously approved these changes Sep 27, 2022
SECURITY.md Outdated Show resolved Hide resolved
SECURITY.md Outdated Show resolved Hide resolved
@jhdalek55
Copy link
Contributor

@hexsecs I corrected a typo and a few other things. I think if the last few corrections are made, we can merge this.

@jhdalek55
Copy link
Contributor

@hexsecs I know we are all busy, but if you can just correct the last few tiny typos, we can merge this.

Co-authored-by: Lois Anne DeLong <[email protected]>
@trishankatdatadog
Copy link
Member

@hexsecs I know we are all busy, but if you can just correct the last few tiny typos, we can merge this.

I committed your changes. Please let us know if this works for you.

Copy link
Member

@trishankatdatadog trishankatdatadog left a comment

Choose a reason for hiding this comment

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

WDYT @jhdalek55 ?

SECURITY.md Outdated Show resolved Hide resolved
@trishankatdatadog trishankatdatadog merged commit 6140243 into uptane:master Oct 14, 2022
@jhdalek55
Copy link
Contributor

Thanks @trishankatdatadog

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.

Add a SECURITY.md security policy to uptane?
7 participants