Skip to content

Blank pentester access configuration while retaining pentester support paths - #2226

Closed
theseanything with Copilot wants to merge 1 commit into
mainfrom
copilot/remove-pentester-access-config
Closed

Blank pentester access configuration while retaining pentester support paths#2226
theseanything with Copilot wants to merge 1 commit into
mainfrom
copilot/remove-pentester-access-config

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown

This PR removes all currently active pentester access configuration from forms-deploy while keeping the existing pentester configuration model and code paths in place for future reuse. It effectively revokes pentester access by setting pentester config to empty defaults rather than deleting feature support.

  • Scope

    • Removes/empties pentester-specific configuration introduced via prior staging updates (IP CIDRs, pentester email entries, and related access-config inputs).
    • Keeps Terraform/Ruby/HCL structures, variables, and validation logic intact.
  • Configuration changes

    • Sets pentester config values to schema-valid empty values ([], {}, or "" as appropriate per field).
    • Applies this consistently in environment-specific config where pentester access was previously configured.
  • What is intentionally unchanged

    • No removal of pentester feature plumbing (modules, variable definitions, or validation mechanisms).
    • No unrelated config or infrastructure behavior changes.

Example of the applied pattern:

# before
pentesters = {
  allowed_ips    = ["203.0.113.10/32", "203.0.113.11/32"]
  allowed_emails = ["pentester1@example.com", "pentester2@example.com"]
}

# after
pentesters = {
  allowed_ips    = []
  allowed_emails = []
}

Copilot AI changed the title [WIP] Remove all current pentester access configuration Blank pentester access configuration while retaining pentester support paths Aug 4, 2026
Copilot AI requested a review from theseanything August 4, 2026 08:46
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