[LO-202] jsoup 이슈 해결

hyuk0309 commented 2 years ago

🛠️ 작업 내용

링크 메타데이터 제목 조회 API에서 예상과 다르게 400 상태 코드를 반환하는 이슈가 있어서 해결했습니다. 원인은 url 정규식이 좁은 범위의 url 포맷만 허용해줘서 그랬습니다.

링크 메타데이터 관련 이슈를 해결하면서 수정하면 더 좋아질 부분들이 보여 제가 추가로 작업하겠습니다.

북마크 엔티티에서 링크 메타데이터 엔티티를 참조하고 있습니다. 그런데 북마크와 링크 메타데이터는 다른 에그리거트이므로 좋지 않은 설계같습니다. 따라서 북마크 엔티티에서 링크 메타데이터 ID만 참조하도록 변경할 예정입니다.
정책적으로 북마크는 무조건 링크 메타데이터 아이디를 외래키로 가져야 합니다. 그런데 생각해보니까 북마크 url에 제대로된 링크 메타데이터가 존재하지 않다면, 의미가 없는 참조같다는 생각이 들었습니다. 이로 인해 링크 메타데이터 테이블에도 필요없는 데이터가 쌓이는 것 같아요. 따라서 북마크에서 링크 메타데이터 아이디를 갖지만 null도 허용하게 바꿀 생각입니다. 추가로 링크 메타데이터 타이틀 조회시에도 링크 메타데이터가 존재하지 않으면 db에 저장하지 않을 생각입니다.

@ndy2 , @jk05018 , @NewEgoDoc, @hyuk0309

github-actions[bot] commented 2 years ago

226 tests 226 :heavy_check_mark: 34s :stopwatch:   71 suites     0 :zzz:   71 files     0 :x:

Results for commit 257460e2.

:recycle: This comment has been updated with latest results.

ndy2 commented 2 years ago

작성 하신 2번 문제 관해서 저도 그 부분에 대한 비즈니스 로직이 현재 북마크 등록 메서드의 주석에만 드러나는 점이 마음에 안들었는데

link_meatadata_id 를 nullable하ㅔ 만들면 해당 로직을 코드로 옮겨내기 수월할 것 같네요