[Enhancement]: Abstract password hashing and validation #15

Merged

je-sendra merged 0 commits from refs/pull/15/head into main

2024-12-26 11:54:18 +00:00

je-sendra commented

2024-12-26 09:57:20 +00:00

(Migrated from autotime-api.vewtech.net)

PR

Fixes #13

Description

The password hashing code was on the AuthService and "hardcoded" to use the BCrypt hashing algorithm.

This PR adds abstraction so the hashing algorithm can be set in the app settings

Added dependencies

Testing

Checklist

My code follows the coding and style conventions for this project
I made a self-evaluation of my code
I commented my code, specially in hard to understand areas
I made the corresponding changes to the documentation
My changes generate no new warnings
I created unit tests for my code

Additional context

# PR ## Related issue Fixes #13 ## Description  The password hashing code was on the AuthService and "hardcoded" to use the BCrypt hashing algorithm. This PR adds abstraction so the hashing algorithm can be set in the app settings ## Added dependencies  ## Testing  ## Checklist - [x] My code follows the coding and style conventions for this project - [x] I made a self-evaluation of my code - [x] I commented my code, specially in hard to understand areas - [ ] I made the corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I created unit tests for my code  ## Additional context

VewDev referenced this pull request from a commit

2025-10-08 13:18:00 +00:00

Merge pull request #15 from VewDev/VewDev/issue13

No reviewers

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

LokiShell/Charwiki!15

No description provided.

Rows
Columns