-
Notifications
You must be signed in to change notification settings - Fork 0
@coderabbitai #20
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
Closed
Closed
@coderabbitai #20
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
44b7dab
feat: change ID generation strategy to UUID for User entity
OtavioXimarelli 2e59fa4
feat: enhance security configuration with stateless session managemen…
OtavioXimarelli f1362f3
feat: add AuthenticationDTO for user login and password encapsulation
OtavioXimarelli 5ba2dab
feat: add AuthenticationController for user login functionality
OtavioXimarelli 2794d25
feat: change ID generation strategy to IDENTITY and add constructor t…
OtavioXimarelli 853ceae
feat: add RegisterDTO for user registration data encapsulation
OtavioXimarelli 7ff4ab7
feat: add user registration endpoint to AuthenticationController
OtavioXimarelli f2f13f5
feat: configure CORS and update security settings for authentication …
OtavioXimarelli 678bd58
fix: improve code formatting and readability in User class
OtavioXimarelli 1fe7d62
fix: reorder import statements for improved clarity in UserRepository
OtavioXimarelli 16eb749
fix: enhance security configuration for authentication endpoints
OtavioXimarelli 54c13b6
fix: remove unused CORS configuration class
OtavioXimarelli 6625f11
fix: remove unused CORS imports from SecurityConfig
OtavioXimarelli 6577982
fix: specify generic type for ResponseEntity in AuthenticationControl…
OtavioXimarelli 8237ca6
fix: update application.properties for add logging level for security
OtavioXimarelli 847fa97
chore: reorganize pom.xml for better structure and readability
OtavioXimarelli 3d834ca
fix: update application.properties to use environment variables for s…
OtavioXimarelli 43e1404
fix: update JWT secret property in application.properties for better …
OtavioXimarelli fc1c33a
fix: load environment variables using Dotenv in AiFoodAppApplication
OtavioXimarelli fdfbead
feat: implement TokenService for JWT generation and validation
OtavioXimarelli 27580bd
feat: add SecurityFilter to handle JWT authentication and user valida…
OtavioXimarelli b51fda7
feat: refactor SecurityConfig to move to security package and integra…
OtavioXimarelli 678200a
feat: add LoginResponseDTO for handling login response data
OtavioXimarelli 4689df1
feat: update AuthenticationController to include JWT token generation…
OtavioXimarelli dbdd6a0
feat: enhance dotenv configuration to set multiple system properties …
OtavioXimarelli 6a431e1
feat: update README to reflect user authentication and Spring Securit…
OtavioXimarelli ae8ba7a
fix: change findByLogin method return type from Optional<UserDetails>…
OtavioXimarelli 6dbb2d1
refactor: simplify user authentication logic in SecurityFilter
OtavioXimarelli fb6daf4
fix: update token generation to use username instead of login and imp…
OtavioXimarelli 0a1cb6d
refactor: remove unused Optional import in UserRepository
OtavioXimarelli 8530eef
fix: update loadUserByUsername method to handle null user return from…
OtavioXimarelli d94b209
feat: add DotenvEnvironmentPostProcessor to spring.factories for envi…
OtavioXimarelli 5f0be49
refactor: remove dotenv configuration from main application class
OtavioXimarelli 6af7d1c
feat: add DotenvEnvironmentPostProcessor for loading environment vari…
OtavioXimarelli 39595d3
refactor: rename LoginResposneDTO to LoginResponseDTO for consistency
OtavioXimarelli 2e5bd9f
fix: correct LoginResponseDTO import and use consistent BCryptPasswor…
OtavioXimarelli e513ba5
Merge branch 'master' into SpringSecurity-implementation
OtavioXimarelli 7e32551
Update src/main/java/com/otavio/aifoodapp/AiFoodAppApplication.java
OtavioXimarelli 259fcb2
refactor: change RegisterDTO from record to class for enhanced flexib…
OtavioXimarelli File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Revisar convenção REST nos endpoints de alimentos
Em vez de usar
/api/foods/create, considerePOST /api/foodspara seguir a convenção sem verbos na URL.🤖 Prompt for AI Agents