Chương 19: Secure Code Review Methodology

// CHUẨN AN TOÀN: Đặt rào chắn kiểm định (Sanitizer) giữa Source và Sink
@GetMapping("/search")
public List search(@RequestParam @Size(max = 50) @Pattern(regexp = "^[a-zA-Z0-9 ]+$") String q) {
    return itemService.findItems(q); // Dữ liệu đã được kiểm duyệt an toàn
}
// Service dùng tham số hóa Parameterized Query
public List findItems(String query) {
    return repo.findByNameContaining(query); // JPA tự động tham số hóa
}

// DANH SÁCH SANITIZER CHUẨN THEO TỪNG LOẠI SINK:
// Cho SQL Sink -> PreparedStatement
// Cho HTML Sink -> HtmlUtils.htmlEscape() hoặc OWASP Java HTML Sanitizer
// Cho File Sink -> Path.normalize() + startsWith(baseDir)
// Cho Process Sink -> Tuyệt đối không nhận chuỗi lệnh từ user, chỉ dùng ID nội bộ

// CHUẨN ZERO TRUST: Mọi ranh giới đều phải xác thực, kể cả giữa các microservice nội bộ
@PostMapping("/internal/api/v1/sync")
@PreAuthorize("hasAuthority('SCOPE_service_internal_write')")
public ResponseEntity syncData(@Valid @RequestBody SyncPayload payload,
                                 @AuthenticationPrincipal Jwt jwt) {
    // Xác thực Mutual TLS (mTLS) hoặc Service-to-Service JWT Token
    return ResponseEntity.ok(syncService.process(payload));
}

// PHIÊN BẢN ĐÃ ĐƯỢC CHẤP THUẬN SAU REVIEW:
public record TransferDto(
    @NotBlank @Pattern(regexp = "^[a-zA-Z0-9_]{3,30}$") String toUser,
    @NotNull @DecimalMin(value = "1000.00") @Digits(integer = 12, fraction = 2) BigDecimal amount
) {}

@PostMapping("/transfer")
public ResponseEntity transfer(@Valid @RequestBody TransferDto dto) {
    accountService.moveMoney(dto.toUser(), dto.amount());
    return ResponseEntity.ok("Transfer successful");
}

// CHUẨN AN TOÀN SPRING SECURITY:
public AuthenticationResponse login(LoginRequest req) {
    // 1. Kiểm tra tỷ lệ thử sai IP / Tài khoản
    rateLimiterService.checkLimit(req.getUsername());
    try {
        Authentication auth = authManager.authenticate(
            new UsernamePasswordAuthenticationToken(req.getUsername(), req.getPassword())
        );
        rateLimiterService.resetLimit(req.getUsername());
        return generateTokens(auth);
    } catch (BadCredentialsException e) {
        rateLimiterService.recordFailedAttempt(req.getUsername());
        throw new InvalidCredentialsException("Tên đăng nhập hoặc mật khẩu không chính xác");
    }
}

// CHUẨN AN TOÀN SPRING SECURITY:
@PostMapping("/api/users/{id}/promote")
@PreAuthorize("hasRole('SUPER_ADMIN')") // Chỉ SuperAdmin mới được cấp quyền
public ResponseEntity promoteUser(@PathVariable Long id, 
                                    @AuthenticationPrincipal UserPrincipal currentUser) {
    auditLogService.logRoleChange(currentUser.getId(), id, "ROLE_ADMIN");
    userService.grantAdminRole(id);
    return ResponseEntity.ok("Promoted successfully");
}

// CHUẨN AN TOÀN ĐƯỢC CHẤP THUẬN SAU REVIEW:
private static final Pattern SAFE_HOST = Pattern.compile("^[a-zA-Z0-9.-]{1,100}$");

public void pingHostSafe(String host) throws IOException, InterruptedException {
    if (!SAFE_HOST.matcher(host).matches()) {
        throw new IllegalArgumentException("Địa chỉ host không hợp lệ");
    }
    // Dùng ProcessBuilder với mảng tham số cố định, không khởi động qua Shell (/bin/sh)
    ProcessBuilder pb = new ProcessBuilder("ping", "-c", "1", host);
    pb.redirectErrorStream(true);
    Process process = pb.start();
    process.waitFor(5, TimeUnit.SECONDS);
}

// CHUẨN AN TOÀN: Đọc từ Secret Manager và cấu hình pre-commit hook
@Service
public class PaymentService {
    private final String stripeSecret;

    public PaymentService(@Value("${stripe.secret.key}") String stripeSecret) {
        this.stripeSecret = stripeSecret;
    }
}

// CHUẨN AN TOÀN: Luôn sử dụng UserSummaryResponse DTO
public record UserPublicProfileDto(
    Long id,
    String displayName,
    String avatarUrl
) {}

@GetMapping("/api/users/{id}")
public ResponseEntity getUser(@PathVariable Long id) {
    User user = userRepository.findById(id).orElseThrow();
    return ResponseEntity.ok(new UserPublicProfileDto(user.getId(), user.getDisplayName(), user.getAvatarUrl()));
}

// CHUẨN AN TOÀN ĐƯỢC PHÊ DUYỆT:
@ExceptionHandler(Exception.class)
public ProblemDetail handleUnexpectedException(Exception ex) {
    String errorTrackingId = UUID.randomUUID().toString();
    // Ghi log chi tiết trên máy chủ để điều tra nội bộ
    log.error("Unhandled exception [TrackingID: {}]", errorTrackingId, ex);

    // Trả về cho Client thông điệp chung an toàn tuyệt đối
    ProblemDetail problem = ProblemDetail.forStatusAndDetail(
        HttpStatus.INTERNAL_SERVER_ERROR,
        "Đã xảy ra sự cố nội bộ. Vui lòng liên hệ hỗ trợ với mã theo dõi: " + errorTrackingId
    );
    problem.setTitle("Internal Server Error");
    return problem;
}