[moin-review] 2021-09-01

Facts

어드민에 신분증 운전면허증 유효성 검사 부분에서 발급지역번호가 최근( 2014년 7월 )에 숫자로 바뀌어 발급지역번호가 한글과 숫자 둘다 가능함을 확인하지 못하고 숫자만 가능하도록 기능을 만들었다.

운전면허증 확인한다고 나무위키와 나 자신의 운전면허증만 확인한 것이 문제였다.

도로교통공단에서 확인했다면 한글, 숫자 모두 가능함을 미리 파악할 수 있었다.

또한 여권 검사하는 부분에서는 client 에서는 문제가 없었지만 마이데이터 허브 쪽에서 여권 확인을 추가해 놓지 않아 문제가 생겼었다.

물론 마이데이터 허브쪽 잘못이 분명히 있지만 미리 확인안한 내 잘못도 있다.

주민번호, 운전면허증만 확인하고 어차피 같은 서버니까 문제 없겠지 라고 판단한게 문제였다.

애초에 주민번호, 운전면허증, 여권 다 확인했으면 운영환경에 배포 전에 문제를 파악해서 조치할 수 있었다.

하지만 그냥 ~~ 겠지 라고 넘기는 바람에 배포를 2~3 번 더해야 했다.

Findings

stackoverflow 나 블로그 같은 소위 확실히 검증되지 않은 문서만 보고 진행하는 건 위험하다.

e2e test 는 할꺼면 확실히 전부 해야한다.

Feelings

전반적으로 신뢰성있는 문서를 기반으로 일을 진행하지 않은게 아쉬웠다.

또한 test 를 할거면 한두개 하고 말게 아니라 확실히 전부 해야한다는 걸 깨달았다..