Skip to content

[PM-35252]#7501

Merged
ike-kottlowski merged 8 commits intomainfrom
auth/pm-35252/update-read-ssouser-sproc
Apr 29, 2026
Merged

[PM-35252]#7501
ike-kottlowski merged 8 commits intomainfrom
auth/pm-35252/update-read-ssouser-sproc

Conversation

@ike-kottlowski
Copy link
Copy Markdown
Contributor

@ike-kottlowski ike-kottlowski commented Apr 18, 2026

🎟️ Tracking

PM-35252

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.71%. Comparing base (53dc0c4) to head (993bed1).
⚠️ Report is 9 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7501      +/-   ##
==========================================
+ Coverage   59.25%   63.71%   +4.46%     
==========================================
  Files        2081     2081              
  Lines       92036    92036              
  Branches     8179     8179              
==========================================
+ Hits        54534    58642    +4108     
+ Misses      35563    31368    -4195     
- Partials     1939     2026      +87     

☔ 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.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 18, 2026

Logo
Checkmarx One – Scan Summary & Detailsb45532bc-5d42-4ea8-9efe-ec9c4ec4adc6


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

@ike-kottlowski ike-kottlowski marked this pull request as ready for review April 23, 2026 16:31
@ike-kottlowski ike-kottlowski requested a review from a team as a code owner April 23, 2026 16:31
@ike-kottlowski ike-kottlowski requested a review from rkac-bw April 24, 2026 12:01
@sonarqubecloud
Copy link
Copy Markdown

@ike-kottlowski ike-kottlowski merged commit 27ae3d5 into main Apr 29, 2026
46 checks passed
@ike-kottlowski ike-kottlowski deleted the auth/pm-35252/update-read-ssouser-sproc branch April 29, 2026 21:19
jaasen-livefront pushed a commit that referenced this pull request Apr 29, 2026
* update sql migrations

* chore: formatting

---------

Co-authored-by: rkac-bw <148072202+rkac-bw@users.noreply.github.com>
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