This repository has been archived by the owner on Aug 16, 2022. It is now read-only.
feat: authentication system #108
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* auth server imp * fix * - fix state parameter - add x509 cert implementation * - add x509 cert to key store - fix request is not updated - fix refresh token flow - handle login error messages - get first key as default if kid is not presented * - fix linting issues * - accept form payload in login request * - code enhancement - fix PR comments * - fix devMode * - give credentials type a better name * - add test for User_GetAuthByProvider * - define custom error for invalid login * - enhance unique list method performance * - fix package alias * - refactor storage init * - use usecase instead of controller - remove unused controller method * - move shared clients into separated file * - handle concurrency in auth storage * - fix go mod * - refactor part of the code * - fix error handling - load Pkix from config * - use dummy DN for the cert - remove unnecessary fields from the cert * - get user info from db * - add missing cn - pass get user func instead of userUseCase * - code enhancement
- remove default key selecting
* - logout api * - remove the unnecessary wrap method - fix http redirection status * - add another logout endpoint for auth0/auth0-react
* feat: create mailer (go-mailer is used) * fix linter * sendGrid implementation * refactor: separate smtp and sendgrid * refactor: add sender name to config * fix newWithSMTP wrong parameter * mixed email content break down MailerConfig struct * resolve notes * resolve notes testing * testing * test cases * test cases * Merge branch 'feat/authentication-system' of https://github.com/reearth/reearth-backend into auth/mailer # Conflicts: # go.mod # internal/app/config.go * Merge branch 'feat/authentication-system' of https://github.com/reearth/reearth-backend into auth/mailer # Conflicts: # go.mod * swap plain and html contents * switch to alternative * remove unused functions * validate email and smtp url
* - Model - Repo * - fix go mod * - move auth storage logic to usecase\interactor * - fix lint issue * - fix PR comments * implement memory storage for authRequest repo
* remove sugnup mutation * new signup logic * merge auth0 and signup logic * lint * revert signup mutation * resolve notes * remove password length * resolve notes * test cases * change comparing method * resolve notes * match password * update user data-layer * feat: password reset (#79) * - password reset * - refactor * - implement the memory repo * - remove unnecessary structs * - fix password reset token validation - add unit test * - fix linting * - fix unit test * - refactor * - replace uuid lib - rename validate method * - fix go mod * - fix go mod * - enhance unit test * - enhance unit test * mock Co-authored-by: rot1024 <aayhrot@gmail.com> * feat: send mail (#86) * - send mail * enhancement * fix log * notes * - add email design * resolve * fix user model * refactor: move to bcrypt * update verify password checks Co-authored-by: yk-eukarya <81808708+yk-eukarya@users.noreply.github.com> Co-authored-by: rot1024 <aayhrot@gmail.com> Co-authored-by: yk <yk.eukarya@gmail.com>
* - save the key and the cert of auth server in mongo db * - fix lint * - make auth srv returns error instead of panicking * - lock config * - refactor - fix memory imp * - fix load method * - fix load method * - add minimum duration to reload config * - fix pr comments * - fix lock logic * fix lint * - fix config repo - use config repo in migrations * - remove unused code * store cert as string * fix string conversion
# Conflicts: # pkg/user/builder.go # pkg/user/builder_test.go # pkg/user/initializer_test.go # pkg/user/user.go
* upgrade to go-jwt-middleware/v2 * omp multi iss * imp multi iss * fix pr Comments
Codecov Report
@@ Coverage Diff @@
## main #108 +/- ##
==========================================
- Coverage 36.95% 36.14% -0.81%
==========================================
Files 304 312 +8
Lines 27329 28447 +1118
==========================================
+ Hits 10099 10283 +184
- Misses 16303 17219 +916
- Partials 927 945 +18
|
* password criteria * fix linter * Update pkg/user/user.go Co-authored-by: yk-eukarya <81808708+yk-eukarya@users.noreply.github.com> * Update pkg/user/user.go Co-authored-by: rot1024 <aayhrot@gmail.com> * fix test * add test case for non-latin characters password Co-authored-by: yk-eukarya <81808708+yk-eukarya@users.noreply.github.com> Co-authored-by: rot1024 <aayhrot@gmail.com>
# Conflicts: # go.mod # go.sum # internal/app/app.go # internal/app/public.go
Co-authored-by: yk-eukarya <81808708+yk-eukarya@users.noreply.github.com> Co-authored-by: yk <yk.eukarya@gmail.com> Co-authored-by: rot1024 <aayhrot@gmail.com>
rot1024
approved these changes
Mar 11, 2022
rot1024
approved these changes
Mar 11, 2022
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
What I've done
What I haven't done
How I tested
Which point do I want you to review particularly
Memo