Skip to content

Update email addresses for pen testers - #2201

Merged
theseanything merged 1 commit into
mainfrom
update-pentester-emails
Jul 13, 2026
Merged

Update email addresses for pen testers#2201
theseanything merged 1 commit into
mainfrom
update-pentester-emails

Conversation

@theseanything

Copy link
Copy Markdown
Contributor

Updated email addresses for pen testers.

Copilot AI review requested due to automatic review settings July 13, 2026 09:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the penetration tester contact email allowlists used by the Forms staging account configuration and the deploy account engineer-access role configuration, ensuring access is granted to the current tester identities.

Changes:

  • Replaced Accenture email addresses with digital.cabinet-office.gov.uk addresses for pentester allowlists in staging account tfvars.
  • Updated the deploy engineer-access pentesters list to match the new pentester email addresses.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
infra/deployments/forms/account/tfvars/staging.tfvars Updates pentester_email_addresses for the staging account configuration.
infra/deployments/deploy/engineer-access/roles.tf Updates the pentesters list passed into the engineer-access module for deploy.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@theseanything
theseanything added this pull request to the merge queue Jul 13, 2026
Merged via the queue into main with commit 62949be Jul 13, 2026
20 checks passed
@theseanything
theseanything deleted the update-pentester-emails branch July 13, 2026 09:08
theseanything added a commit that referenced this pull request Aug 4, 2026
The penetration test has finished, so clear the pentester email
addresses, CIDR ranges and @accenture.com sign-in domain added in #2196,
#2201 and #2202.
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.

5 participants