검색 필드의 허용 목록, 유지보수할 때 남겨 둘 예제
클라이언트가 보낸 정렬 열이나 필터 이름을 SQL에 그대로 붙이면 데이터 노출과 쿼리 오류 위험이 커진다. 다시 읽을 수 있는 최소 맥락를 중심으로 작은 예제와 확인 기준을 정리합니다.
클라이언트가 보낸 정렬 열이나 필터 이름을 SQL에 그대로 붙이면 데이터 노출과 쿼리 오류 위험이 커진다.
시간이 지나면 왜 이렇게 구현했는지보다 어떻게 실행하는지만 남기기 쉽다. 옵션과 제약을 잊은 채 코드를 정리하면 의도했던 동작까지 바뀔 수 있다. 이번에는 작은 예제를 실행 가능한 설명으로 남기고, 구현이 달라져도 유지할 부분을 분리한다.
다시 읽을 수 있는 최소 맥락
외부 필드 이름을 허용된 내부 열에 명시적으로 매핑한다. 값은 바인딩하고 필드와 연산자도 허용 목록에서 선택하도록 한다.
이번에 확인할 질문: 내부 구조를 바꾸더라도 이 예제가 설명하는 결과는 그대로 유지되어야 하는가?
작게 확인하는 예제
다음 예제는 핵심 조건을 드러내기 위한 출발점이다. 서비스의 전체 구현을 대신하지 않으므로 실제 입력 범위와 실패 처리 규칙을 함께 읽어야 한다.
allowed_sort = {'publishedAt': Post.published_at, 'title': Post.title}
field = request_sort
if field not in allowed_sort:
raise ValueError('unsupported sort field')
statement = select(Post).order_by(allowed_sort[field])
허용하지 않은 열 이름과 연산자를 전달해 명확하게 거부되고 내부 열이나 관계가 노출되지 않는지 확인한다.
놓치기 쉬운 조건
값을 바인딩했다고 식별자나 연산자까지 자동으로 안전해지는 것은 아니다. ORM을 사용하더라도 임의 문자열로 SQL 구조를 만들지 않도록 한다.
| 확인할 부분 | 살펴볼 내용 |
|---|---|
| 기준 동작 | 허용하지 않은 열 이름과 연산자를 전달해 명확하게 거부되고 내부 열이나 관계가 노출되지 않는지 확인한다. |
| 비교 조건 | 입력과 실행 순서를 고정하고 한 번에 한 조건만 바꾼다. |
| 적용 범위 | 예제가 생략한 의존성과 실제 서비스의 제약을 구분한다. |
다음 변경에 적용할 순서
- 구현 세부 사항보다 외부에서 관찰하는 결과를 적는다.
- 예제가 의존하는 입력과 환경을 가까이에 둔다.
- 설명이 실제 동작과 달라지면 코드와 예제를 같은 변경에서 고친다.
오래 남는 문서는 가장 긴 문서가 아니라 다시 확인할 수 있는 문서다. 선택의 이유와 작동하는 예제가 함께 있으면 새로운 구현으로 옮길 때도 기준을 잃지 않는다. 중복 설명을 늘리기보다 정본이 어디인지 분명히 하는 편이 유지 비용을 줄인다.
이 이야기가 도움이 되었나요?