Merge pull request #22 from lumijiez/security_branch

Security branch
This commit was merged in pull request #22.
This commit is contained in:
DmitriiKaban
2023-10-25 11:54:17 +03:00
committed by GitHub
2 changed files with 4 additions and 6 deletions

View File

@@ -41,7 +41,7 @@ public class ExpenseController {
}
@PostMapping()
public ResponseEntity<ExpenseDTO> createNewExpense(@RequestBody ExpenseCreationDTO expenseDTO,
public ResponseEntity<Void> createNewExpense(@RequestBody ExpenseCreationDTO expenseDTO,
BindingResult bindingResult) {
Expense expense = expenseMapper.toExpense(expenseDTO);
@@ -54,8 +54,7 @@ public class ExpenseController {
expense.setUser(user);
expenseService.createOrUpdate(expense);
ExpenseDTO createdExpenseDTO = expenseMapper.toDto(expense);
return ResponseEntity.status(HttpStatus.CREATED).body(createdExpenseDTO);
return ResponseEntity.status(HttpStatus.CREATED).build();
}
return ResponseEntity.notFound().build();

View File

@@ -41,7 +41,7 @@ public class IncomeController {
}
@PostMapping()
public ResponseEntity<IncomeDTO> createNewIncome(@RequestBody IncomeCreationDTO incomeDTO,
public ResponseEntity<Void> createNewIncome(@RequestBody IncomeCreationDTO incomeDTO,
BindingResult bindingResult) {
Income income = incomeMapper.toIncome(incomeDTO);
Authentication authentication = SecurityContextHolder.getContext().getAuthentication();
@@ -54,8 +54,7 @@ public class IncomeController {
System.out.println(income);
incomeService.createOrUpdate(income);
IncomeDTO createdIncomeDTO = incomeMapper.toDto(income);
return ResponseEntity.status(HttpStatus.CREATED).body(createdIncomeDTO);
return ResponseEntity.status(HttpStatus.CREATED).build();
}
return ResponseEntity.notFound().build();