-
Notifications
You must be signed in to change notification settings - Fork 0
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
Authentication #22
Merged
Merged
Authentication #22
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
…om error messages, hash passwords before saving them to database and prevent returning the password to the client side
…lude <header> and <nav> elements correspondingly; make the app name in Header redirect to the HomePage; create a route for authentication and include it in Navbar
… the data is still loading or an error was catched
…is.com and sets it into localStorage when fulfilled, add it to Redux store
…he newly created getUser action (authSlice) to enable this feature
…the localStorage so that the user stays logged in until he logs out
…ned in alonside with the Sing Out button; for non-authenticated users only Login button is shown
… (signup/signin), test the feature in Postman
…hSlice and dispatching them in Auth component
…ent process data coming both from google-oauth or regular signin/signup
…array), update components and forms to exlude unnecessary arrays
… it from req.params instead of req.body since we can't access req.body in delte requests
…ce (signin/signup forms)
…ing to refresh the page after registration
…en there are no users in db
…are refetched with error or refetching is in process
… token to client since later the server will decode them differently
…e to import it to all Redux slices; interceptor provides the Authorization token depending on the token type
…ract the userId, we'll later use it to query data and restrict some actions
… otherwise users have to refresh the page after registration to see the pages that require credentials
…t the user information
…the oauth access_token can not be decoded by jwt)
Closed
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Features developed in this branch:
Client: token is passed using axios interceptors.
Server: token is verified in
auth
middlevare.User credentials are stored in Cookies and expire in 1 h. The user gets logged out automatically after Cookies expire.
Some small adjustments and bugfixes were performed as well.