Skip to content

Initial write-up for member audit#100

Draft
RRosio wants to merge 5 commits into
jupyter:mainfrom
RRosio:member_audit
Draft

Initial write-up for member audit#100
RRosio wants to merge 5 commits into
jupyter:mainfrom
RRosio:member_audit

Conversation

@RRosio

@RRosio RRosio commented Mar 25, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@manics

manics commented Mar 27, 2025

Copy link
Copy Markdown
Contributor

I think this makes sense. JupyterHub kind-of has a policy for following up on inactive members, though I can't remember the last time we followed it: https://jupyterhub-team-compass.readthedocs.io/en/latest/practices/check-ins.html

What might be worth doing in this policy is emphasizing that GitHub privileges are not a reflection of project membership or status within that project. Traditionally it's been seen that way, but from a security perspective you might be one of the most important/influential people in a project, but if your role doesn't require GitHub privileges then you shouldn't have them.

@Carreau

Carreau commented Mar 27, 2025

Copy link
Copy Markdown
Member

Sorry i'm going to stay short.
I think a user can/should be able to be marked manually as active (with a reason) on any tooling.

GitHub is uppercase G and H I believe.

@rpwagner

rpwagner commented Apr 1, 2025

Copy link
Copy Markdown
Contributor

This is looking good. Let's make the first goal to get this to into this repo and communicate it. Next, I'd like to propose a subsecurity section in the governance documents under Organizational Policy.

Learning from this and the 2FA implementation, we can probably draft a short guide on how to propose, decide, communicate, and implement future security policies.

@dlqqq dlqqq left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@RRosio Thank you for putting this together! This is a great first start. We can make more revisions later. Just one minor comment worth addressing here. 🤗

Comment thread docs/member-auditing.md

## Reinstatement of Access

If a member's privileges are adjusted due to inactivity, they can be reinstated upon request. Our goal is to maintain security without hindering future contributions. No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As a reader, it's unclear how I should reach out to the Security Council to reinstate my access privileges. We should probably suggest a way to contact us here, for example:

Suggested change
If a member's privileges are adjusted due to inactivity, they can be reinstated upon request. Our goal is to maintain security without hindering future contributions.
If a member's privileges are adjusted due to inactivity, they can be reinstated on request via email to `security@ipython.org`. Our goal is to maintain security without hindering future contributions.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this be handled via the subproject instead of the central security email?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes! This isn't about a security vulnerability so having it go to the mailing list is better--and more likely to be seen.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Perhaps! This ambiguity is exactly why I think we should document it here first. 😁

Comment thread docs/member-auditing.md Outdated
@JosephTLucas

Copy link
Copy Markdown

@RRosio the audit log documentation might help you find elements for the inactive criteria since that's probably how we'll automate.

Co-authored-by: David L. Qiu <david@qiu.dev>
@RRosio

RRosio commented Apr 22, 2025

Copy link
Copy Markdown
Contributor Author

In today's security meeting, we proposed handling a more detailed review of membership levels as a follow-up, after first addressing inactive members.

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.

6 participants