8000 Feature/change password by Ratsch0k · Pull Request #126 · Ratsch0k/group-car-api · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Feature/change password #126

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 52 commits into from
Mar 10, 2022
Merged

Feature/change password #126

merged 52 commits into from
Mar 10, 2022

Conversation

Ratsch0k
Copy link
Owner
@Ratsch0k Ratsch0k commented Mar 9, 2022

No description provided.

@Ratsch0k
Added documentation of the new endpoint
64228bb
@Ratsch0k
Improved logging for createValidationResultHandler
b5217fa
@Ratsch0k
Added first prototype for an authentication service
f89a52e
@Ratsch0k
Added waitOnLogin property to config
4da4133
@Ratsch0k
Added utility function to create a router for handling validations
49c7457
@Ratsch0k
Added missing properties to db config which could lead to a broken da…
085c657 
…tabase configuration
@Ratsch0k
Added IncorrectPasswordError
df4ff19
@Ratsch0k
Added NewPasswordMustBeDifferentError
b51c340
@Ratsch0k
Adapt to name change of bcrypt config
72facb5
@Ratsch0k
Changed UserNotFoundError to also accept username
469e9e0
@Ratsch0k
Added findByUsername and findById to UserRepository
b5d360b
@Ratsch0k
Added utility function to bind arguments to debug logging functions
97de707
@Ratsch0k
Adapt to name change of bcrypt to auth
220b0dd
@Ratsch0k
Added default environment vars to db.yml
b97ea82
@Ratsch0k
Added .env to dockerignore and added empty .env to docker image
fe23231
@Ratsch0k
Added email to Express.User
25e06fa
@Ratsch0k
Small changes to Database class
603351c
@Ratsch0k
Added new errors to index file
7034de1
@Ratsch0k
Changed convertUserToJWTPayload to accept UserDto
63cd66e
@Ratsch0k
Added checkPassword and changePassword methods to UserService
69b4225
@Ratsch0k
Added service.ts in which common types for all services are defined
0e79cdc
@Ratsch0k
Added index file for auth folder
e83233e
@Ratsch0k
Improved SignUpController to use async/await
6b9813c
@Ratsch0k
Aggregated validators for user properties into own file
181e3b2
@Ratsch0k
Use createValidationRouter in user-id-validator.ts
7b8af23
@Ratsch0k
Added createValidationRouter in sign-up-validator.ts
ef9b6c3
@Ratsch0k
Added controller for changing a user's password
9bc2134
@Ratsch0k
Added necessary routes for new endpoint
61c5784
@Ratsch0k
Removed link from docker-compose.dev.yml to tmp folder of host
aecd463
@Ratsch0k
Changed default port to 8000
fc53b27
Ratsch0k added 22 commits March 7, 2022 20:22
@Ratsch0k
Added check to changePassword to check if new password is different f…
1025b5c 
…rom old one
@Ratsch0k
Small change in logging of signUpController
47000cc
@Ratsch0k
Changed status code on successful password change to 204
fc33a94
@Ratsch0k
Fixed errors in authentication-service.ts
1de966b
@Ratsch0k
Changed test database to be groupcarapitest
fbb7e67
@Ratsch0k
Added missing comments in user-service.ts
01d52bb 
Comments were missing for following:
 - checkPassword
 - changePassword
@Ratsch0k
Added missing comments in UserRepository for findById
6892d8e
@Ratsch0k
Made deletedAt in Express.User optional
2d2af7b
@Ratsch0k
Added logging at start of changePasswordController
9a5fb41
@Ratsch0k
Added validation router for changePassword route
bab7639
@Ratsch0k
Fixed tests for signUpController
c27a797
@Ratsch0k
Added test utility function for easier integration test setup
0bc0295
@Ratsch0k
Added unit and integration tests for change-password functionality
f4d4fb0
@Ratsch0k
Fixed changePassword not rethrowing unexpected errors
e9222a0
@Ratsch0k
Added unit tests for checkPassword and changePassword
76c0409
@Ratsch0k
Added unit tests for findById in UserRepository
dbbcf01
@Ratsch0k
Fixed incorrect handling of transaction in signUpController
44810d3
@Ratsch0k
Fixed unit tests for signUpController
8499d27
@Ratsch0k
Fixed linting issue
8b1fa22
@Ratsch0k
Fixed unnecessary whitespace in openapi.yaml
f22ec00
@Ratsch0k
Fixed code smell in Dockerfile
829505c
@Ratsch0k
Fixed other code smells in Dockerfile
444841a
@Ratsch0k Ratsch0k merged commit f38e1bc into develop Mar 10, 2022
@Ratsch0k Ratsch0k deleted the feature/change-password branch March 10, 2022 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Ratsch0k
0