Azure-Samples / Azure-Samples/Copilot-Studio-with-Azure-AI-Search

[Task] Remove unnecessary Checkov suppressions and document remaining ones

Open
#229 0 comments 1 reaction 1 assignee Assigned to @Copilot View on GitHub
infra terraform
Dominant language
HCL
Stars
31
Forks
8
PR merge metrics
No merged PRs in 30d

Description

### Description

Review the repository for existing Checkov suppressions, particularly those related to NSG warnings, as these issues seem to have been addressed. Remove any Checkov suppressions that are no longer needed. For any suppressions that must remain, clearly document:

- The reason each suppression is still required
- Guidance on what steps would be necessary to harden the resource or remove the warning in the future

This will help ensure the codebase remains secure and maintainable, and that any deviations from best practices are well-justified and actionable.

You can validate if suppressions are still needed by running script `azd-hooks/scripts/hooks/preprovision/run_checkov.ps1` and seeing if checkov fails without it.

### Definition of Done

```markdown
- suppressions removed
- checkov scan succeeded
- remaining suppressions documented with comments,
- comments say how to harden and link to security considerations doc
- remaining suppressions are documented in security considerations
```

### Values Provided

More secure solution.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.