Skip to content

brokers: use providerID for user identification - #1546

Merged
denisonbarbosa merged 11 commits into
mainfrom
use-sub-for-identification
Jun 30, 2026
Merged

brokers: use providerID for user identification#1546
denisonbarbosa merged 11 commits into
mainfrom
use-sub-for-identification

broker: correct the offline cache migration comment in finishAuth

cd574c9
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Jun 30, 2026 in 1s

87.25% (-0.74%) compared to ea62880

View this Pull Request on Codecov

87.25% (-0.74%) compared to ea62880

Details

Codecov Report

❌ Patch coverage is 81.57895% with 77 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.25%. Comparing base (ea62880) to head (cd574c9).
⚠️ Report is 15 commits behind head on main.

Files with missing lines Patch % Lines
authd-oidc-brokers/internal/broker/broker.go 79.54% 45 Missing ⚠️
...d-oidc-brokers/internal/dbusservice/dbusservice.go 0.00% 10 Missing ⚠️
internal/services/pam/pam.go 75.75% 8 Missing ⚠️
internal/users/db/migration.go 70.00% 6 Missing ⚠️
internal/users/manager.go 88.37% 5 Missing ⚠️
internal/users/db/update.go 94.11% 1 Missing ⚠️
internal/users/db/users.go 94.11% 1 Missing ⚠️
pam/internal/adapter/brokerselection.go 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1546      +/-   ##
==========================================
- Coverage   87.98%   87.25%   -0.74%     
==========================================
  Files          99      120      +21     
  Lines        6832     8315    +1483     
  Branches      111      111              
==========================================
+ Hits         6011     7255    +1244     
- Misses        765     1004     +239     
  Partials       56       56              

☔ View full report in Codecov by Harness.
📢 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.