Skip to content

Update README.md #1528

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MohammedAdam240604
Copy link

feat: Add user authentication system

Adds a new user authentication system to the application, including user registration, login, and password reset functionality.

This change addresses a long-standing request to secure user data and enable personalized user experiences. The implementation uses JWTs for session management and Argon2 for secure password hashing. New endpoints have been added at /api/auth/register, /api/auth/login, and /api/auth/reset. A new AuthService was created to handle all authentication logic.

The new system requires a JWT_SECRET environment variable to be set. It also introduces a minor breaking change: users must now be logged in to access previously public profile pages.

feat: Add user authentication system

Adds a new user authentication system to the application, including user registration, login, and password reset functionality.

This change addresses a long-standing request to secure user data and enable personalized user experiences. The implementation uses JWTs for session management and Argon2 for secure password hashing. New endpoints have been added at `/api/auth/register`, `/api/auth/login`, and `/api/auth/reset`. A new `AuthService` was created to handle all authentication logic.

The new system requires a `JWT_SECRET` environment variable to be set. It also introduces a minor breaking change: users must now be logged in to access previously public profile pages.
@MohammedAdam240604
Copy link
Author

MohammedAdam240604 commented Aug 9, 2025

"Hi team,

This PR addresses the issue of [briefly state the problem or feature].

The key change is the introduction of [new function/class/method] in the [file-name] file, which refactors the logic for [explain the refactoring]. I've also added a new test to ensure all user data endpoints are functioning as expected.

I believe this change improves [state the benefit, e.g., readability, performance, or security]. I've tested it locally, and all checks are passing.

Please let me know if you have any feedback! Thanks!"

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.

1 participant