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

Do project self-assessment #451

Closed
baentsch opened this issue Jul 18, 2024 · 14 comments
Closed

Do project self-assessment #451

baentsch opened this issue Jul 18, 2024 · 14 comments
Assignees

Comments

@baentsch
Copy link
Member

As per CNCF

@baentsch baentsch self-assigned this Jul 18, 2024
@baentsch
Copy link
Member Author

First cut below. It would be great if you could take a look and maybe provide actionable feedback @anvega . Most things are pretty obvious but I'm feeling an ethical obligation to first witness more committed contributors before implementing/declaring as "good" things this self-assessment suggests. Otherwise, I'd be afraid this would create a false sense of reliability to users ("badges", "alliance endorsement", etc marketing fluff) -- all the while the code is maintained thanklessly by the proverbial random guy in Nebraska (err, Switzerland :).

oqsprovider-self-assessment-20240726.md

@baentsch
Copy link
Member Author

@anvega Thanks for taking up this initial self-assessment. @dstebila by now corrected my incomplete understanding and reference as to the responsibilities of the LF triage team and I'll need to update that section once I better understand how it's been set up by LF.

@ryjones : Can you shed some light on this? Who gets notified if someone "privately" reports a vulnerability via GH? Is this going to the maintainers? Is (handling that) part of the undocumented responsibilities of LF maintainers? Who else gets these notifications? You as LF admin?

@ryjones
Copy link

ryjones commented Jul 30, 2024

@baentsch:

Organization administrators, repository administrators, and teams with the security manager role

These teams are security managers:

@baentsch
Copy link
Member Author

Thanks for the explanation, @ryjones . This seems a bit broad and completely contradicting the public documentation indicating a "responsibly small" set (only admins) to see this:
image

Created open-quantum-safe/tsc#60 to track/improve. Tagging @dstebila @hartm as meeting follow-up.

@anvega : Updated oqsprovider-self-assessment-20240731.md to properly document this.

@ryjones
Copy link

ryjones commented Jul 31, 2024

The project can make that list as large or small as they like. If you want to create a specific team for security, you can do that, and I'll replace the current list with that one.

@anvega
Copy link

anvega commented Aug 2, 2024

Thanks @baentsch and @ryjones.

I've converted the markdown to Google Docs for easier commenting. I've also completed my first pass with some initial naive questions. As a recommendation through the assessment, I suggest rephrasing the text where questions might arise, to preemptively address them rather than discussing them in the comments. This approach ensures that when we convert back to markdown, the content is there

A few more people have shown interest in participating in the review. I'm waiting for formal approval from @JustinCappos to initiate the joint assessment process. Once approved, I’d like to give the reviewers a chance to ask their initial questions before we convene for a group discussion with you the project team, and the other reviewers.

@baentsch
Copy link
Member Author

baentsch commented Aug 2, 2024

I've also completed my first pass with some initial naive questions.

How/where do I see those questions? How could I comment on them? I see the Google doc, but can't seem to change "Mode" or "Comments".

@baentsch
Copy link
Member Author

baentsch commented Aug 2, 2024

Oh, and the underlying document is the wrong (old) version. Please use the new version labelled 20240731 attached above, @anvega .

@baentsch
Copy link
Member Author

baentsch commented Aug 2, 2024

Now I understood that you used my personal email address for this (getting edit access). Please don't do that for such "public documents": I don't want to receive spam in there. In the order of preference, please use my github ID, my public github email address (57787676+baentsch@...) or the private one I created only for GH interactions (info@....). I guess when that's changed I'll also get to see questions and can respond.

@baentsch
Copy link
Member Author

baentsch commented Aug 2, 2024

Thanks, @anvega -- problem's resolved.

@anvega
Copy link

anvega commented Aug 2, 2024

Apologies for the mix up with the document versions. I've now updated the Security Issue Resolution section to reflect the changes in the latest version.

I've also removed your personal email address and used the info@... address for Google access since the GH one wasn't accepted by Google.

Regarding your inability to see the comments, you should be able to view them even with just viewing access. Try clicking on View > Comments > Expand all comments. If this doesn't work, I can relay the questions to you via another method, such as a markdown file.

@baentsch baentsch mentioned this issue Aug 18, 2024
4 tasks
@dehatideep
Copy link

@baentsch I have added my Qs in the same doc, please see and clarify. Thank you.

@baentsch
Copy link
Member Author

@dehatideep Thanks for your questions. Please check whether my answers in the doc are clear enough or whether I need to provide more background.

@baentsch
Copy link
Member Author

Assessment complete. Created #483 to track actual (interest for) improvements to the code.

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

No branches or pull requests

4 participants