Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Feb 15, 2025

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)가 삭제되는 오류 수정
1 participant