Skip to content

WAS finding scan conversion linked to CRASM-2492#832

Merged
cduhn17 merged 22 commits intodevelopfrom
cd_was_scan_conversion
Jul 22, 2025
Merged

WAS finding scan conversion linked to CRASM-2492#832
cduhn17 merged 22 commits intodevelopfrom
cd_was_scan_conversion

Conversation

@cduhn17
Copy link
Collaborator

@cduhn17 cduhn17 commented Mar 31, 2025

🗣 Description

Creates a scan that pulls/update/insert Qualys data. This scan is the same scan from ATC framework and integrated into XFD

💭 Motivation and context

We want to bring all data gathering into the XFD repo. WAS is our scan used by P&E and will give the XFD dashboard a new type of Qualys data to display

🧪 Testing

Tested Locally

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All future TODOs are captured in issues, which are referenced
    in code comments.
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.

@cduhn17 cduhn17 self-assigned this Mar 31, 2025
@cduhn17 cduhn17 changed the title WAS finding scan conversion WAS finding scan conversion linked to CRASM-2322 Mar 31, 2025
@cduhn17 cduhn17 changed the title WAS finding scan conversion linked to CRASM-2322 WAS finding scan conversion linked to CRASM-2492 Jun 9, 2025
@cduhn17 cduhn17 marked this pull request as ready for review July 11, 2025 14:18
schmelz21
schmelz21 previously approved these changes Jul 15, 2025
@cduhn17 cduhn17 requested a review from schmelz21 July 22, 2025 18:20
Copy link
Contributor

@nickviola nickviola left a comment

Choose a reason for hiding this comment

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

LGTM

@cduhn17 cduhn17 merged commit 9e31078 into develop Jul 22, 2025
20 of 21 checks passed
@cduhn17 cduhn17 deleted the cd_was_scan_conversion branch July 22, 2025 18:44
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.

4 participants