Skip to content

fix: remove panic() and relay password hashing errors to UI - #1014

Merged
eternal-flame-AD merged 1 commit into
masterfrom
bcrypt-errors
Aug 10, 2026
Merged

fix: remove panic() and relay password hashing errors to UI#1014
eternal-flame-AD merged 1 commit into
masterfrom
bcrypt-errors

fix: remove panic() and relay password hashing errors to UI

75bc19c
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Aug 9, 2026 in 1s

74.28% (-0.25%) compared to d543a8a

View this Pull Request on Codecov

74.28% (-0.25%) compared to d543a8a

Details

Codecov Report

❌ Patch coverage is 58.97436% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.28%. Comparing base (d543a8a) to head (75bc19c).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
api/user.go 50.00% 8 Missing and 4 partials ⚠️
auth/password/password.go 75.00% 1 Missing and 1 partial ⚠️
database/database.go 50.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1014      +/-   ##
==========================================
- Coverage   74.53%   74.28%   -0.25%     
==========================================
  Files          66       66              
  Lines        3483     3512      +29     
==========================================
+ Hits         2596     2609      +13     
- Misses        688      698      +10     
- Partials      199      205       +6     

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