Skip to content

PM-32518 Create new V2 endpoints for SummaryData within Access Intelligence#7048

Closed
prograhamming wants to merge 8 commits intomainfrom
dirt/pm-32518-summary-data-endpoints-v2-access-intelligence
Closed

PM-32518 Create new V2 endpoints for SummaryData within Access Intelligence#7048
prograhamming wants to merge 8 commits intomainfrom
dirt/pm-32518-summary-data-endpoints-v2-access-intelligence

Conversation

@prograhamming
Copy link
Contributor

@prograhamming prograhamming commented Feb 20, 2026

🎟️ Tracking

This PR is a subtask (PM-32518) for user story PM-31923

📔 Objective

Creating new V2 endpoints for read and update operations on the SummaryData field in the database.

📸 Screenshots

@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026

Logo
Checkmarx One – Scan Summary & Detailse5d9d9fa-181d-4441-a78d-0105cae5b308


New Issues (2) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 MEDIUM CSRF /src/Api/Dirt/Controllers/OrganizationReportsV2Controller.cs: 90
detailsMethod at line 90 of /src/Api/Dirt/Controllers/OrganizationReportsV2Controller.cs gets a parameter from a user request from request. This parame...
Attack Vector
2 MEDIUM CSRF /src/Api/Dirt/Controllers/OrganizationReportsV2Controller.cs: 90
detailsMethod at line 90 of /src/Api/Dirt/Controllers/OrganizationReportsV2Controller.cs gets a parameter from a user request from request. This parame...
Attack Vector

Fixed Issues (1) Great job! The following issues were fixed in this Pull Request
Severity Issue Source File / Package
MEDIUM CSRF /src/Api/Vault/Controllers/CiphersController.cs: 293

@prograhamming prograhamming marked this pull request as ready for review February 20, 2026 19:16
@prograhamming prograhamming requested a review from a team as a code owner February 20, 2026 19:16
@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

❌ Patch coverage is 88.67925% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.79%. Comparing base (5b20ee9) to head (e27650b).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...irt/Controllers/OrganizationReportsV2Controller.cs 80.39% 0 Missing and 10 partials ⚠️
...rts/ReportFeatures/AddOrganizationReportCommand.cs 0.00% 0 Missing and 1 partial ⚠️
...atures/UpdateOrganizationReportSummaryV2Command.cs 95.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7048      +/-   ##
==========================================
+ Coverage   56.76%   56.79%   +0.03%     
==========================================
  Files        2014     2018       +4     
  Lines       88218    88321     +103     
  Branches     7855     7874      +19     
==========================================
+ Hits        50073    50165      +92     
  Misses      36321    36321              
- Partials     1824     1835      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@theMickster theMickster left a comment

Choose a reason for hiding this comment

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

Will you please remove all of the region blocks?
While I understand we still have them around the code base, they are explicitly requested to not be used in our contributing docs and we have programmed Claude not to use them. Thank you!

contributing docs

@sonarqubecloud
Copy link

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