DAT-19181 GitHub Actions Error: uploading S3 Object to Bucket (#285)

* ✨ (send-docs-redirects-to-staging.yml): add a check for website_redirect format to ensure all values start with a forward slash for consistency and correctness in redirect paths * 📝 (variables.tf): add a comment to indicate the need for a list of redirects from team members to clarify requirements * 📝 (send-docs-redirects-to-staging.yml): improve validation for website_redirect format to ensure it starts with '/' or 'http' 💡 (variables.tf): remove unnecessary trailing whitespace for cleaner code * 📝 (variables.tf): add a comment placeholder for obtaining redirect list from team members * ♻️ (variables.tf): update redirect path to remove leading slash for consistency in URL formatting * ♻️ (variables.tf): update redirect path to include leading slash for consistency and correctness in URL structure * ✨ (send-enterprise-redirects-to-staging.yml): add a step to check the format of website_redirect entries to ensure valid URLs 💡 (send-enterprise-redirects-to-staging.yml): provide error messages and examples for invalid website_redirect formats to guide users
liquibase · Dec 12, 2024 · b48b79b · b48b79b
1 parent 2c1bc44
commit b48b79b
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 0 deletions.
diff --git a/.github/workflows/send-docs-redirects-to-staging.yml b/.github/workflows/send-docs-redirects-to-staging.yml
@@ -41,6 +41,23 @@ jobs:
           role-to-assume: ${{ secrets.AWS_ADMIN_GITHUB_OIDC_ROLE_ARN_DOCS }}
           aws-region: us-east-1
 
+      - name: Check website redirects format
+        run: |
+          INVALID_REDIRECTS=$(grep -P 'website_redirect\s*=\s*"[^/h]|website_redirect\s*=\s*"h(?!ttps?://)' variables.tf || true)
+          if [ ! -z "$INVALID_REDIRECTS" ]; then
+            echo "❌ Error: Found website_redirect entries with invalid format:"
+            echo "----------------------------------------"
+            echo "$INVALID_REDIRECTS" | while read -r line; do
+              echo -e "\033[31m$line\033[0m"
+            done
+            echo "----------------------------------------"
+            echo "website_redirect must start with either '/' or 'http'"
+            echo "Examples:"
+            echo "  website_redirect = \"/home.html\""
+            echo "  website_redirect = \"https://example.com\""
+            exit 1
+          fi
+
       - name: Terraform Format
         if: ${{ github.event_name == 'pull_request' }}
         id: fmt

diff --git a/.github/workflows/send-enterprise-redirects-to-staging.yml b/.github/workflows/send-enterprise-redirects-to-staging.yml
@@ -38,6 +38,23 @@ jobs:
           role-to-assume: ${{ secrets.AWS_ADMIN_GITHUB_OIDC_ROLE_ARN_DOCS }}
           aws-region: us-east-1
 
+      - name: Check website redirects format
+        run: |
+          INVALID_REDIRECTS=$(grep -P 'website_redirect\s*=\s*"[^/h]|website_redirect\s*=\s*"h(?!ttps?://)' variables.tf || true)
+          if [ ! -z "$INVALID_REDIRECTS" ]; then
+            echo "❌ Error: Found website_redirect entries with invalid format:"
+            echo "----------------------------------------"
+            echo "$INVALID_REDIRECTS" | while read -r line; do
+              echo -e "\033[31m$line\033[0m"
+            done
+            echo "----------------------------------------"
+            echo "website_redirect must start with either '/' or 'http'"
+            echo "Examples:"
+            echo "  website_redirect = \"/home.html\""
+            echo "  website_redirect = \"https://example.com\""
+            exit 1
+          fi
+
       - name: Terraform Format
         if: ${{ github.event_name == 'pull_request' }}
         id: fmt