Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use modules for sensitive WAF config #1666

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

theseanything
Copy link
Contributor

@theseanything theseanything commented Jan 29, 2025

This uses a private module to create privates WAF resources and set private variables. These modules are defined in https://github.com/alphagov/terraform-govuk-infrastructure-sensitive (annoying had to use this name convention for TFC) .

As part of the tfc-configuration workspace, a sensitive variable set for WAFs is created for each environments.

Then for each of the govuk-publishing-infrastructure projects, they are configured to use the respective variable-set and to create the private WAF resource.

(tfc-configuration project needs to be run twice; once to create the variable set, then to assign it to the relevant projects. Hence will error on review of this PR.)

@samsimpson1
Copy link
Member

Could this be a use case for the private module registry? https://developer.hashicorp.com/terraform/cloud-docs/registry/publish-modules

That might be a bit complicated though

@theseanything theseanything force-pushed the use-modules-for-sensitive-resources branch 13 times, most recently from fcb4621 to c488a09 Compare January 30, 2025 16:55
@theseanything theseanything force-pushed the use-modules-for-sensitive-resources branch from c488a09 to 8646b36 Compare January 30, 2025 17:16
@theseanything theseanything force-pushed the use-modules-for-sensitive-resources branch from 48ce053 to a3ad192 Compare January 30, 2025 17:35
@theseanything theseanything requested review from samsimpson1 and a team January 31, 2025 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants