Skip to content

SEAL Certifications References#300

Draft
Robert-MacWha wants to merge 9 commits intodevelopfrom
certs-refs
Draft

SEAL Certifications References#300
Robert-MacWha wants to merge 9 commits intodevelopfrom
certs-refs

Conversation

@Robert-MacWha
Copy link
Collaborator

@Robert-MacWha Robert-MacWha commented Nov 27, 2025

Add references to existing frameworks within cert controls where relevant.

Also took notes of all the areas that are distinctly missing refs, attached as a txt file since it's fairly long. We can create sub-issues for the most critical tasks.

  • Secure Development Environments #273
  • Comprehensive account / access control framework
    • Write a canonical section on account / access control for web2, EX account authorization (2fa, yubikey, single-sign-on, etc), password management, permission management (EX aws permissions) .
  • Comprehensive monitoring page
    • Write a canonical monitoring page. Probably need to split it by on-chain vs off-chain targets, with the on-chain one being more important imo (since it'll be the same minus implementation details across a wider set of topics, while off-chain is more reliant on implementation specifics).
  • Incident response for compromised multisig / treasury / wallet.
    • Referenced in both multisig + treasury op certs, and would be generally helpful. I'm sure it exists.
  • Workspace Security
    • Basically need to write the whole thing.

certs-notes.txt

@vercel
Copy link

vercel bot commented Nov 27, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
frameworks Ready Ready Preview, Comment Dec 17, 2025 10:24pm

@mattaereal
Copy link
Collaborator

Can you pull from develop and resolve conflicts? Also @DicksonWu654 can you review this?

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 1, 2025

Deploying frameworks with  Cloudflare Pages  Cloudflare Pages

Latest commit: e888cbb
Status: ✅  Deploy successful!
Preview URL: https://6b7a4939.frameworks-573.pages.dev
Branch Preview URL: https://certs-refs.frameworks-573.pages.dev

View logs

@mattaereal mattaereal added the content:update This issue or PR updates content or suggests to label Dec 15, 2025
@mattaereal
Copy link
Collaborator

I don't know why nobody commented on this already, or why nobody flagged what's happening here 😅 . Just ask for help or tag me if you're stuck, folks <3.

This is a bigger issue, not that it will be complicated to solve. But by using mdx files to add certs, we're essentially breaking gray-matter reads YAML files, since title is mentioned several times, and it is expected to find only one. Will work on a patch for this

Add error handling for frontmatter parsing.
@DicksonWu654
Copy link
Collaborator

Ah sorry - I put my comments in the discord a while ago but my problem w the current references is that they reference things that are completely relevant / complete for the cert requirement.

@mattaereal
Copy link
Collaborator

I understand, there's no issue with that. We'll need to troubleshoot it so we can keep it as it is.

@mattaereal
Copy link
Collaborator

I'm working on it btw. I was serving and forgot to run just build instead of serving, that's why I was missing this last error.

@mattaereal mattaereal self-assigned this Dec 17, 2025
@mattaereal
Copy link
Collaborator

Found the culprits. The mdx files have an intertwined title and description, and in some cases, the title is duplicated below, sandwiching the description. I'll remove them manually. I don't think there's a reason to keep them, and I assume you're not referring to them when you say "they reference things that are completely relevant / complete for the cert requirement."

@Robert-MacWha
Copy link
Collaborator Author

Ah good catch Matta - looks like a merge conflict from 0a492cb.

@DicksonWu654 I assume you meant "irrelevant / incomplete"? (a) I made a lot of the changes you suggested so it should be better, and (b) you're welcome to make a PR changing anything remaining you don't like.

@mattaereal
Copy link
Collaborator

Ok folks, this should build now. If this was spoken via Discord, I'm sorry I missed it. I even tried looking for it, but just found a debate on the certs channel, not specifically about this failing. Make sure to always add others to review it, and assign it to the steward. And if the steward gets stuck, they are instructed to look for help elsewhere!

@mattaereal
Copy link
Collaborator

Moving this to draft, since it seems to be undergoing a debate between steward @DicksonWu654 and contributor @Robert-MacWha. Let us know when it is ready to be published, or go ahead and do it yourselves after achieving resolution

@mattaereal mattaereal marked this pull request as draft December 22, 2025 16:07
@scode2277
Copy link
Collaborator

Hey guys, any info on this PR?

@DicksonWu654
Copy link
Collaborator

DicksonWu654 commented Jan 12, 2026 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content:update This issue or PR updates content or suggests to

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants