Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Feb 15, 2025

#572

📄 요약(Summary)

DTO 데이터 전송 시 개행 문자(\n)가 삭제되는 오류 수정

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • Jsoup.clean(value, Safelist.basic())이 HTML 태그뿐만 아니라 개행 문자 같은 일부 특수 문자도 필터링하여, dto 데이터에 대해 개행 문자도 삭제하여 데이터 전송이 되는 오류가 발생하였습니다.
  • 해당 부분을 Jsoup.parse(value).wholeText()으로 대체하여 모든 HTML 태그를 제거하고,개행(\n)과 공백을 유지하도록 수정하였습니다.
image

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

  • Jsoup.clean(value, Safelist.basic())이 HTML 태그뿐만 아니라 개행 문자 같은 일부 특수 문자도 필터링하고 있었음.
  • Jsoup.parse(value).wholeText()을 사용하여 모든 HTML 태그를 제거하고,개행(\n)과 공백을 유지핳 수 있음.

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #572

@joowojr joowojr self-assigned this Feb 15, 2025
@joowojr joowojr added the 🐛 bug 버그를 수정합니다 label Feb 15, 2025
@joowojr joowojr merged commit c83a08b into develop Feb 15, 2025
1 check passed
@joowojr joowojr deleted the CLAP-438 branch February 15, 2025 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug 버그를 수정합니다

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-438 DTO 데이터 전송 시 개행 문자(\n)가 삭제되는 오류 수정

2 participants