2023-08-21 정기 백엔드 개발 회의

[scarf005]

안건을 자유롭게 추가해주세요.

[scarf005]

배포 일정에 대해

버그픽스가 포함돼 있어 #675 를 빨리 머지하면 좋을 것 같습니다.

[jimin52]

지금 배포가 안 되고 있는데 codeQL 친구 무시하고 머지할 방법이 없을까요?

[scarf005]

codeQL리뷰는 생략 가능하나 리뷰 approve가 없어 머지가 되지 않는 것으로 보입니다.

[scarf005]

완료했습니다!

[jimin52]

rate-limit 미들웨어를 추가하는 이슈 생성

code scanner 가 경고한 dos 공격에 대비한 rate-limit 미들웨어의 도입 필요한 것 같습니다.

참고 자료
해당 이슈

[scarf005]

개발 서버에서 하는 방법처럼 nginx에 rate limit을 거는 방식도 괜찮을 것 같습니다.

backend/nginx.dev/nginx.conf

Line 28 in 6f1957b

limit_req_zone $binary_remote_addr zone=api:10m rate=1r/s;

backend/nginx.dev/conf.d/default.conf

Line 40 in 6f1957b

limit_req zone=api burst=5;

참고 링크

• https://www.nginx.com/blog/rate-limiting-nginx/
• https://willseungh0.tistory.com/191

[jimin52]

책 조회의 경우 타자를 칠때마다 api 콜이 날아오는 것으로 알고 있습니다.
CRUD 중 Read 와 CUD 의 rate limit 을 따로 걸던가
하나로 통합한다고 해도 모든 rate limit 을 넉넉하게 걸어놓는 것이 좋을 것 같습니다.
기본값으로 초당 10회정도가 좋을 것 같아요

[scarf005]

네, nginx 라우트에서 경로별로 rate limit해주면 될 것 같습니다.

별개로, 검색창 기능을 프론트엔드에서 처리하는 방식도 DB 쿼리 수 감소 측면에서 고려해 볼 만해 보입니다. 프론트엔드에서 매일 indexedDB에 전체 도서 목록을 캐싱해 두었다가, 매 타자마다 캐싱한 목록에서 fuzzy search를 하는 방식으로요. 다만 아직은 백엔드에서 검색을 처리하는 것이 더 간단할 것 같네요.