-
Couldn't load subscription status.
- Fork 599
test: Add future year CVE antipattern for testing #14922
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
base: abadawi/multi-spec-radar
Are you sure you want to change the base?
Conversation
2d2dc49 to
9205e9f
Compare
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
🟢 Challenge Submitted by @abadawi591Finding: Feedback:
Challenge ID: |
9205e9f to
5801083
Compare
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
5801083 to
709d2bf
Compare
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
Pipeline should now fetch token from mariner-pipelines-kv/cblmarghGithubPRPat No ADO pipeline variable needed - single source of truth!
51d82f1 to
3c4e77e
Compare
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
This confirms: - No ADO pipeline variable needed - Token fetched from Key Vault using Managed Identity - Single source of truth: mariner-pipelines-kv/cblmarghGithubPRPat
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
- Testing challenge system with third antipattern - Outdated CVE from 2020 being patched in 2025 (should be flagged) - Tests analytics categorization with multiple issue types - Release bumped to 7
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
🟢 Challenge Submitted by @abadawi591
Issue: Feedback from @abadawi591:
Challenge ID: |
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
🟢 Challenge Submitted by @abadawi591
Issue: Feedback from @abadawi591:
Challenge ID: |
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
🟢 Challenge Submitted by @abadawi591
Issue: Feedback from @abadawi591:
Challenge ID: |
- Use querySelector within modal container as fallback for finding child elements - getElementById may fail for elements inside display:none containers - Add modal innerHTML logging for debugging when elements are missing - This should resolve the 'Modal child elements missing' error
📊 Interactive HTML Report🔗 CLICK HERE to open the Interactive HTML ReportThe report will open in a new tab automatically Features:
🔴 CVE Spec File Check - FAILEDOverall Severity: 🔴 ERROR 📋 Executive Summary
📦 Package Analysis Details
|
🟢 Challenge Submitted by @abadawi591
Issue: Feedback from @abadawi591:
Challenge ID: |
- Log modal element details (tagName, id, childCount, innerHTML length) - Log innerHTML content (first 500 chars) to see actual HTML structure - Test both getElementById and querySelector methods separately - Count all span elements in modal as fallback diagnostic - This will help identify why modal child elements are not found
Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-staticsubpackages, etc.) have had theirReleasetag incremented../cgmanifest.json,./toolkit/scripts/toolchain/cgmanifest.json,.github/workflows/cgmanifest.json)./LICENSES-AND-NOTICES/SPECS/data/licenses.json,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)*.signatures.jsonfilessudo make go-tidy-allandsudo make go-test-coveragepassSummary
What does the PR accomplish, why was it needed?
Change Log
Does this affect the toolchain?
YES/NO
Associated issues
Links to CVEs
Test Methodology