From b97a3480d5380b22ce0bbeaf4dcf4b4b2f2dff4c Mon Sep 17 00:00:00 2001
From: Connor Catlett <conncatl@amazon.com>
Date: Thu, 12 Sep 2024 18:12:27 +0000
Subject: [PATCH] Add permissions to all GitHub actions

Signed-off-by: Connor Catlett <conncatl@amazon.com>
---
 .github/workflows/generate-code-coverage.yaml | 5 +++++
 .github/workflows/govulncheck.yaml            | 3 +++
 .github/workflows/helm-chart-release.yaml     | 4 ++++
 .github/workflows/output-code-coverage.yaml   | 4 ++++
 .github/workflows/release.yaml                | 4 ++++
 .github/workflows/trivy-containers.yaml       | 4 ++++
 .github/workflows/trivy.yaml                  | 4 ++++
 .github/workflows/unit-tests.yaml             | 3 +++
 8 files changed, 31 insertions(+)

diff --git a/.github/workflows/generate-code-coverage.yaml b/.github/workflows/generate-code-coverage.yaml
index 9ac03b674..fbc5240a1 100644
--- a/.github/workflows/generate-code-coverage.yaml
+++ b/.github/workflows/generate-code-coverage.yaml
@@ -14,6 +14,11 @@
 
 name: Generate Code Coverage
 on: [pull_request]
+
+permissions:
+  contents: read
+# upload-artifact does not need write permissions as it relies on an undocumented token
+# https://github.com/actions/upload-artifact/issues/197#issuecomment-832279436
   
 jobs:
   cover-base:
diff --git a/.github/workflows/govulncheck.yaml b/.github/workflows/govulncheck.yaml
index c0046391f..d783655ba 100644
--- a/.github/workflows/govulncheck.yaml
+++ b/.github/workflows/govulncheck.yaml
@@ -15,6 +15,9 @@
 name: 'govulncheck'
 on: [pull_request]
 
+permissions:
+  contents: read
+
 jobs:
   govulncheck:
     runs-on: ubuntu-latest
diff --git a/.github/workflows/helm-chart-release.yaml b/.github/workflows/helm-chart-release.yaml
index 78e6ab96e..2e46cf1a2 100644
--- a/.github/workflows/helm-chart-release.yaml
+++ b/.github/workflows/helm-chart-release.yaml
@@ -21,6 +21,10 @@ on:
     paths:
       - "charts/**/Chart.yaml"
 
+permissions:
+  contents: write
+  pages: write
+
 jobs:
   release:
     runs-on: ubuntu-latest
diff --git a/.github/workflows/output-code-coverage.yaml b/.github/workflows/output-code-coverage.yaml
index 7411d0fd8..0cbd19de8 100644
--- a/.github/workflows/output-code-coverage.yaml
+++ b/.github/workflows/output-code-coverage.yaml
@@ -17,6 +17,10 @@ on:
   workflow_run:
     workflows: [Generate Code Coverage]
     types: [completed]
+
+permissions:
+  contents: read
+  pull-requests: write
   
 jobs:
   output-code-coverage:
diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml
index 49fa87786..a3c418eb7 100644
--- a/.github/workflows/release.yaml
+++ b/.github/workflows/release.yaml
@@ -18,6 +18,10 @@ on:
     # Sequence of patterns matched against refs/tags
     tags:
       - "v*" # Push events to matching v*, i.e. v1.0, v20.15.10
+
+permissions:
+  contents: write
+
 jobs:
   build:
     name: Release
diff --git a/.github/workflows/trivy-containers.yaml b/.github/workflows/trivy-containers.yaml
index ca024e77b..50803690b 100644
--- a/.github/workflows/trivy-containers.yaml
+++ b/.github/workflows/trivy-containers.yaml
@@ -20,6 +20,10 @@ on:
   schedule:
     - cron: '0 */24 * * *'
 
+permissions:
+  contents: read
+  security-events: write
+
 jobs:
   build-matrix:
     runs-on: ubuntu-latest
diff --git a/.github/workflows/trivy.yaml b/.github/workflows/trivy.yaml
index e5a40300e..261c411d3 100644
--- a/.github/workflows/trivy.yaml
+++ b/.github/workflows/trivy.yaml
@@ -18,6 +18,10 @@ on:
     branches:
       - master
   pull_request:
+
+permissions:
+  contents: read
+  security-events: write
   
 jobs:
   build:
diff --git a/.github/workflows/unit-tests.yaml b/.github/workflows/unit-tests.yaml
index 123a2b8c3..e3d8dcc97 100644
--- a/.github/workflows/unit-tests.yaml
+++ b/.github/workflows/unit-tests.yaml
@@ -18,6 +18,9 @@ on:
     branches:
       - master
   pull_request:
+
+permissions:
+  contents: read
   
 jobs:
   buildx: