Skip to content

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

Open
eternal-flame-AD wants to merge 1 commit into
masterfrom
bcrypt-errors
Open

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

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

38eccee
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Jul 28, 2026 in 0s

74.33% (-0.20%) compared to d543a8a

View this Pull Request on Codecov

74.33% (-0.20%) compared to d543a8a

Details

Codecov Report

❌ Patch coverage is 57.69231% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.33%. Comparing base (d543a8a) to head (38eccee).

Files with missing lines Patch % Lines
api/user.go 40.00% 6 Missing and 3 partials ⚠️
database/database.go 50.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1014      +/-   ##
==========================================
- Coverage   74.53%   74.33%   -0.20%     
==========================================
  Files          66       66              
  Lines        3483     3499      +16     
==========================================
+ Hits         2596     2601       +5     
- Misses        688      695       +7     
- Partials      199      203       +4     

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