Skip to content

Security Documentation#49

Open
maennchen wants to merge 1 commit intohexpm:mainfrom
maennchen:sdlc-docs
Open

Security Documentation#49
maennchen wants to merge 1 commit intohexpm:mainfrom
maennchen:sdlc-docs

Conversation

@maennchen
Copy link
Copy Markdown
Contributor

Add Security Documentation around Hex.pm.

Comment thread security/README.md
Comment on lines +39 to +44
### [Operations](./operations/incident-response.md)

How we operate and respond to security events.

- [Incident Response](./operations/incident-response.md) - Triage, response, notification
- [Access Control](./operations/access-control.md) - Internal access principles
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I did not have much to go on here.

@ericmj Could we collaborate to make this accurate?

Comment thread security/sdlc/runtime.md

### Production

<!-- TODO: Verify production secrets management -->
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

TODO

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I can add you to the hexpm-ops that holds our terraform, kubernetes, and fastly configuration if that would help verification.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That would be great, yes.


### Access Control

<!-- TODO: Verify least privilege implementation -->
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

TODO

Copy link
Copy Markdown
Member

@ericmj ericmj left a comment

Choose a reason for hiding this comment

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

This is really great, thank you! I added a couple of comments and questions.

Comment thread security/sdlc/risk-register.md Outdated
Comment thread security/sdlc/runtime.md

### Production

<!-- TODO: Verify production secrets management -->
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I can add you to the hexpm-ops that holds our terraform, kubernetes, and fastly configuration if that would help verification.

Comment thread security/sdlc/runtime.md Outdated

## Deployment Controls

Deployments are performed manually by core maintainers outside of GitHub infrastructure.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Our deployments happens through CI on hexpm-ops. We also have a Slack bot that can deploy, monitor deployments, and monitor container image builds. That's on https://github.com/hexpm/hexpm_deploy, I can add you to that too.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes please

Comment thread security/supply-chain/overview.md
Comment thread security/supply-chain/provenance.md Outdated
Comment thread security/threat-model/assumptions.md Outdated
Comment thread security/threat-model/client-flows.md
Comment thread security/threat-model/client-flows.md
Dev-->>Client: TOTP code
else Has API Key
Note over Client: Use API key directly
opt 2FA Enabled on Account
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You cannot use 2FA with API key

Client->>Client: Read repo config from mix.exs
Client->>Client: Identify private repos (org: "mycompany")

Note over Client,API: Authenticate for private repo
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we show OAuth flow here also?

Comment thread security/threat-model/client-flows.md Outdated
Comment thread security/threat-model/client-flows.md Outdated
Comment thread security/threat-model/assets.md Outdated
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.

2 participants