보안 취약점에 집중한 코드 리뷰

코드가 하는 일

1. 쇼핑몰에서 내보낸 orders.csv를 읽습니다.
2. 주문 줄마다 stock.xlsx에서 SKU로 상품을 찾아 수량을 뺍니다.
3. 바뀐 재고를 다시 저장하고 합계를 import_log.txt에 기록합니다.

구조가 명확하고 함수 이름도 잘 지어져 있습니다. 문제 대부분은 스크립트가 예상하지 못한 형태의 입력이 들어올 때 생깁니다.

버그와 취약점

번호	줄	심각도	내용
1	48	높음	같은 CSV를 두 번 가져올 수 있음. 이미 처리한 주문 번호를 확인하지 않음. 재고가 마이너스가 되는 가장 유력한 원인
2	72	높음	환불 주문의 빈 값이나 "2.0" 같은 값에서 int(row["qty"])가 오류로 멈춤
3	95	중간	SKU 비교가 대소문자를 구분해서 "gt-101"과 "GT-101"을 다른 상품으로 처리
4	130	중간	encoding="utf-8" 없이 파일을 열어 윈도우에서 한글 상품명이 깨짐
5	201	낮음	맨 except:가 실제 오류를 숨겨 로그가 쓸모없어짐

수정 코드(일부)

문제 1: 이미 가져온 주문 건너뛰기

processed = load_processed_ids("processed_orders.txt")
for row in reader:
    if row["order_id"] in processed:
        continue
    # ... 기존 처리 로직 ...
    processed.add(row["order_id"])
save_processed_ids("processed_orders.txt", processed)

이유: 같은 파일로 스크립트를 다시 실행해도 주문은 한 번만 반영됩니다.

문제 2: 수량을 안전하게 읽기

raw = (row.get("qty") or "").strip()
try:
    qty = int(float(raw))
except ValueError:
    log.warning("주문 %s: 잘못된 수량 %r, 건너뜀", row["order_id"], raw)
    continue

이유: 환불이나 부분 취소 줄은 전체 작업을 멈추지 않고, 알아보기 쉬운 기록을 남긴 채 건너뜁니다.

문제 3: SKU 정규화

sku = row["sku"].strip().upper()

재고 딕셔너리를 만들 때도 같은 .strip().upper()를 적용하세요.

스타일과 가독성

• process()가 90줄입니다. read_orders, apply_to_stock, write_report로 나누세요.
• row[7]처럼 숫자로 적은 열 번호는 헤더 이름으로 바꾸세요.

꼭 확인하세요: 각 수정 사항은 파일 사본으로 먼저 테스트하세요. 이번 리뷰는 코드를 읽고 진행했으며 실행은 하지 않았습니다.