Skip to content

Commit 836a398

Browse files
feat(controller): enhance FoodItemController with improved logging and error handling
1 parent 1319fe3 commit 836a398

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

src/main/java/com/otavio/aifoodapp/controller/FoodItemController.java

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,15 @@
3030

3131
@RestController
3232
@RequestMapping("/api/foods")
33-
@CrossOrigin(origins = {"http://localhost:5173", "https://aifoodapp.site"}, allowCredentials = "true")
33+
@CrossOrigin(
34+
origins = {"http://localhost:5173", "https://aifoodapp.site"},
35+
allowCredentials = "true",
36+
allowedHeaders = {"Authorization", "Content-Type", "X-Requested-With", "Accept", "Origin", "X-XSRF-TOKEN"},
37+
exposedHeaders = {"X-XSRF-TOKEN"}
38+
)
3439
public class FoodItemController {
40+
private static final org.slf4j.Logger log = org.slf4j.LoggerFactory.getLogger(FoodItemController.class);
41+
3542
private final FoodItemService foodItemService;
3643
private final FoodMapper foodMapper;
3744

@@ -51,12 +58,12 @@ public FoodItemController(FoodItemService foodItemService, FoodMapper foodMapper
5158
@PostMapping("/create")
5259
public ResponseEntity<?> create(@Valid @RequestBody FoodItemCreateDto createDto) {
5360
try {
54-
System.out.println("Received create food request: " + createDto.getName());
61+
log.info("Received create food request: {}", createDto.getName());
5562
FoodItem foodItem = foodMapper.map(createDto);
5663
FoodItem savedItem = foodItemService.saveWithAiEnhancement(foodItem);
5764
return ResponseEntity.status(HttpStatus.CREATED).body(foodMapper.map(savedItem));
5865
} catch (Exception e) {
59-
e.printStackTrace();
66+
log.error("Error creating food item: {}", e.getMessage(), e);
6067
// Return error details for debugging
6168
return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).body(Map.of(
6269
"error", e.getMessage(),
@@ -83,7 +90,7 @@ public ResponseEntity<List<FoodDto>> list() {
8390
return ResponseEntity.ok(foodDtos);
8491
} catch (Exception e) {
8592
// Log the exception details for debugging
86-
e.printStackTrace();
93+
log.error("Error listing food items: {}", e.getMessage(), e);
8794
throw e;
8895
}
8996
}
@@ -124,7 +131,7 @@ public ResponseEntity<?> testAuth() {
124131
));
125132
}
126133
} catch (Exception e) {
127-
e.printStackTrace();
134+
log.error("Error testing auth: {}", e.getMessage(), e);
128135
return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).body(Map.of(
129136
"error", e.getMessage(),
130137
"type", e.getClass().getName()

0 commit comments

Comments
 (0)