{"id":8074,"date":"2022-01-18T15:46:42","date_gmt":"2022-01-18T15:46:42","guid":{"rendered":"https:\/\/emtunc.org\/blog\/?p=8074"},"modified":"2022-01-18T15:46:44","modified_gmt":"2022-01-18T15:46:44","slug":"github-security-2022-branch-protection-edition","status":"publish","type":"post","link":"https:\/\/emtunc.org\/blog\/01\/2022\/github-security-2022-branch-protection-edition\/","title":{"rendered":"GitHub Security 2022: Branch Protection Edition"},"content":{"rendered":"\n

In this post, I will be going through some of the essential<\/em> Branch Protection Rules you should have on all* of your GitHub repositories. GitHub is constantly releasing new updates all the time<\/a> but my recommendations stand as of January 2022.<\/p>\n\n\n\n

I’ve also added some gotchas<\/em> I’ve tripped over<\/s> discovered whilst deploying such rules across nearly a thousand repositories.<\/p>\n\n\n\n

*Assuming we’re talking about corporate GitHub organisations… and by all, I mean ALL. Branch protection Rules should be the rule, not the exception. Have exceptions if you need them (you will!) but otherwise apply it everywhere.<\/p>\n\n\n\n\n\n\n\n

Which branches to protect?<\/h2>\n\n\n\n

At the very least I’d suggest protecting your main branch; it’s probably called master<\/em> or main<\/em>.<\/p>\n\n\n\n

Protection Rules<\/h2>\n\n\n\n
\"\"<\/a><\/figure>\n\n\n\n

You almost certainly don’t want anyone merging directly to your master branch. It’s worth mentioning that this isn’t strictly only<\/em> a security control; pull requests (PR’s) encourage healthy, secure and reliable codebases by pulling in appropriate team members to review changes to potentially critical services. Talking of reviews…<\/p>\n\n\n\n

\"\"<\/a><\/figure>\n\n\n\n

You should check this box to enforce reviews for proposed code changes to the repository. The number of approvals<\/em> required is largely dependent on SDLC maturity and criticality of the code in the repository.<\/p>\n\n\n\n

Note: If your GitHub Actions Workflow permissions are set to read and write and<\/em> you only require one approval, an adversary can trivially bypass this rule no matter how strict your other protection rules might be.
GitHub approvals are designed such that an author cannot approve their own PR’s, however, a compromised GitHub account can trigger a GitHub Action to create a PR (the PR will be created under a bot user) then approve it with the compromised account.<\/p>\n\n\n\n

\"\"<\/a><\/figure>\n\n\n\n

Closely linked to the above; we want at least one reviewer but we don’t want just anyone<\/em> reviewing and approving the code. We may want a specific team within the GitHub organisation to review and approve the code changes.
Even if you don’t mind anyone<\/em> viewing and approving code changes, I would still recommend having a CODEOWNERS<\/code> file with a team that everyone is a member of, bar service and machine accounts.<\/p>\n\n\n\n

Note: CODEOWNERS<\/code> failures are silent so I’m going to shamelessly plug one of my GitHub repositories<\/a> to help discover such silent failures<\/p>\n\n\n\n

\"\"<\/a><\/figure>\n\n\n\n

It’s usually a good idea to dismiss an approval if additional commits are made after the fact. This helps protect against seemingly harmless but reliability affecting code changes to more malicious ones caused by compromised accounts.<\/p>\n\n\n\n

\"\"<\/a><\/figure>\n\n\n\n

This is a new option that only got released a few months ago but is a big win for security. Previously, the only way to get any form of automation to work on a repository was to uncheck the “Include Administrators” option which meant that any<\/em> repository admin or<\/em> organisation owner could bypass branch protection rules for that repository.<\/p>\n\n\n\n

With this new option, we can apply all branch protection rules to everyone (including admins and owners!) and<\/em> explicitly an automation user which is permitted to bypass the branch protection rules.<\/p>\n\n\n\n

\"\"<\/a><\/figure>\n\n\n\n

Now that the option of allowing specified actors to bypass PR requirements<\/em> exists, this option must<\/em> be checked. I can’t think of any valid reasons not to at this point. Leave a comment if there’s an edge case I haven’t considered!<\/p>\n\n\n\n

\"\"<\/a><\/figure>\n\n\n\n

It’s a good idea to enable status checks that help enable your engineering team. Status checks can help maintain and improve the security and reliability of your services.<\/p>\n\n\n\n

Requiring branches to be up to date is more of a hygiene thing and is more likely to cause reliability issues than anything else if not checked.<\/p>\n\n\n\n

\"\"<\/a><\/figure>\n\n\n\n

This one might be a little controversial (feel free to leave a comment with your thoughts!) but I actually don’t recommend enabling this unless<\/em> your organisation is already heavily invested and embedded in the GPG world (my condolences). <\/p>\n\n\n\n

This control adds very little (it can help mitigate impersonation attacks within the GitHub organisation) but comes at the heavy cost of creating, storing, distributing and managing GPG keys.<\/p>\n\n\n\n

When SSH signing becomes a thing then commit signing starts to sound a lot more attractive. Given SSH signing was released to Git late last year, I am hopeful that we will see it supported at some point in 2022!<\/p>\n\n\n\n

If you’re interested in reading more about commit signing and what attacks it can (and can’t!) defend again, check out Dan Lorenc’s blog where he explains it a lot more eloquently.<\/a><\/p>\n","protected":false},"excerpt":{"rendered":"

In this post, I will be going through some of the essential Branch Protection Rules you should have on all* of your GitHub repositories. GitHub is constantly releasing new updates all the time but my recommendations stand as of January 2022. I’ve also added some gotchas I’ve tripped over discovered whilst deploying such rules across […]<\/p>\n","protected":false},"author":32,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"templates\/template-full-width.php","format":"standard","meta":{"jetpack_post_was_ever_published":false,"footnotes":""},"categories":[1],"tags":[],"class_list":["post-8074","post","type-post","status-publish","format-standard","hentry","category-tech"],"jetpack_featured_media_url":"","jetpack_shortlink":"https:\/\/wp.me\/p1trTO-26e","jetpack_sharing_enabled":true,"_links":{"self":[{"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/posts\/8074","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/users\/32"}],"replies":[{"embeddable":true,"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/comments?post=8074"}],"version-history":[{"count":12,"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/posts\/8074\/revisions"}],"predecessor-version":[{"id":8099,"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/posts\/8074\/revisions\/8099"}],"wp:attachment":[{"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/media?parent=8074"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/categories?post=8074"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/emtunc.org\/blog\/wp-json\/wp\/v2\/tags?post=8074"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}