Skip to content

feat(buckets): add website field to enable GCS static website hosting#16

Merged
drewmercer-bl merged 2 commits intomainfrom
feat/website-static-hosting
Feb 26, 2026
Merged

feat(buckets): add website field to enable GCS static website hosting#16
drewmercer-bl merged 2 commits intomainfrom
feat/website-static-hosting

Conversation

@drewmercer-bl
Copy link
Copy Markdown
Contributor

Summary

  • Adds optional website field to the buckets variable (defaults to false, no breaking change)
  • Passes website through to the terraform-module-backend-bucket submodule call in main.tf
  • When website = true, the GCS bucket is configured with main_page_suffix = index.html and not_found_page = index.html, allowing the load balancer to serve the SPA correctly instead of returning an XML bucket listing

Test plan

  • Set website = true on a bucket in terraform.tfvars and run terraform plan to verify the backend-bucket module receives the argument
  • Apply and confirm https://greenroom.brandlive-sandbox.com/ serves the app instead of XML
  • Confirm buckets without website set continue to behave as before (defaults to false)

🤖 Generated with Claude Code

Adds an optional 'website' boolean to the buckets variable (defaults to
false). When true, passes website=true to the backend-bucket submodule,
which configures the GCS bucket with main_page_suffix=index.html so the
load balancer serves index.html instead of returning an XML listing.
@drewmercer-bl drewmercer-bl force-pushed the feat/website-static-hosting branch from b7c1ed4 to 72ae419 Compare February 26, 2026 19:42
@drewmercer-bl drewmercer-bl merged commit be047ab into main Feb 26, 2026
2 checks passed
@drewmercer-bl drewmercer-bl deleted the feat/website-static-hosting branch February 26, 2026 19:52
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants