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

chore: url 후처리 로직 추가 #84

Merged
merged 1 commit into from
Jul 26, 2024
Merged

Conversation

Marades
Copy link
Collaborator

@Marades Marades commented Jul 26, 2024

PR 내용

파싱한 thumbnail URL의 엣지케이스에 대응하기 위한 후처리 로직 추가

example

  • protocol-relative url
    • 프로토콜에 영향을 안받게끔 URL에서 프로토콜(http: 또는 https:)을 명시하지 않고, 현재 페이지의 프로토콜을 자동으로 따라가도록 하는 방법
      image

추가 및 변경 사항

API endpoint와 추가 혹은 변경된 사항을 적어주세요!

PR 중점사항

리뷰어가 중점적으로 봐야하는 부분에 대해 적어주세요!

스크린샷

@github-actions github-actions bot added etc Chore Branch document 문서화 관련 작업 수정 및 생성 labels Jul 26, 2024
@Marades Marades self-assigned this Jul 26, 2024
Copy link
Collaborator

@J-Hoplin J-Hoplin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

처음에 상희누나가 올려준거 보고 정규표현식 생각했는데, 미미하겠지만 메모리적으로 regex 표현식 검사하는게 더 많이 들꺼같당 ㅋㅋㅋㅋㅋ 이게 더 좋은 방법인거같아 고마워 형!!

@Marades Marades merged commit fb9049c into develop Jul 26, 2024
3 checks passed
@J-Hoplin J-Hoplin deleted the chore/protocol-relative-url branch July 28, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document 문서화 관련 작업 수정 및 생성 etc Chore Branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants