Skip to content

[PM-33788] EF Emergency Access Query Updates#7297

Merged
trmartin4 merged 3 commits intomainfrom
auth/pm-33788/ef-emergency-access
Apr 5, 2026
Merged

[PM-33788] EF Emergency Access Query Updates#7297
trmartin4 merged 3 commits intomainfrom
auth/pm-33788/ef-emergency-access

Conversation

@enmande
Copy link
Copy Markdown
Contributor

@enmande enmande commented Mar 24, 2026

🎟️ Tracking

PM-33788

📔 Objective

Update Emergency Access EF queries for consistency and add tests. See ticket for details.

@enmande enmande changed the title Auth/pm 33788/ef emergency access [PM-33788] EF Emergency Access Query Updates Mar 24, 2026
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

Logo
Checkmarx One – Scan Summary & Detailscbd410d4-d252-4d02-b16e-6295ea49ca6f


New Issues (1) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 MEDIUM SSL_Verification_Bypass /src/Core/Platform/Mail/Delivery/MailKitSmtpMailDeliveryService.cs: 84
details/src/Core/Platform/Mail/Delivery/MailKitSmtpMailDeliveryService.cs relies HTTPS requests, in . The ServerCertificateValidationCallback parameter,...
Attack Vector

Fixed Issues (1) Great job! The following issues were fixed in this Pull Request
Severity Issue Source File / Package
MEDIUM SSL_Verification_Bypass /src/Core/Platform/Mail/Delivery/MailKitSmtpMailDeliveryService.cs: 81

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.05%. Comparing base (783fd5a) to head (52fa8a0).
⚠️ Report is 79 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7297      +/-   ##
==========================================
+ Coverage   57.74%   62.05%   +4.30%     
==========================================
  Files        2043     2043              
  Lines       89868    89885      +17     
  Branches     7991     7989       -2     
==========================================
+ Hits        51895    55774    +3879     
+ Misses      36115    32171    -3944     
- Partials     1858     1940      +82     

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

@enmande enmande marked this pull request as ready for review March 25, 2026 00:48
@enmande enmande requested a review from a team as a code owner March 25, 2026 00:48
@enmande enmande requested a review from ike-kottlowski March 25, 2026 00:48
@trmartin4 trmartin4 removed the needs-qa label Apr 5, 2026
@trmartin4 trmartin4 merged commit 93604aa into main Apr 5, 2026
51 of 54 checks passed
@trmartin4 trmartin4 deleted the auth/pm-33788/ef-emergency-access branch April 5, 2026 18:43
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.

3 participants