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

[버그] 코멘트 작성할 때 링크 주소가 괄호로 끝날경우 링크가 깨지는 현상. #266

Open
E-nuri opened this issue Aug 21, 2017 · 7 comments

Comments

@E-nuri
Copy link
Contributor

E-nuri commented Aug 21, 2017

현재상태

이슈에 코멘트를 작성하기 위해 링크를 첨부하고자 함.
링크의 끝부분에 괄호가 있어서 링크가 제대로 완성되질 않고 깨진상태로 생성됨.
(두번째 사진에서 하이퍼링크가 먹은 파란 글자와 달리 검정색으로 된 글자)

2017-08-21 8 51 05

2017-08-21 8 51 14

문제점

URL을 코멘트 작성 후 생성된 링크를 클릭했을때 아래 이미지처럼 변환된다.
2017-08-21 9 01 42

참고링크
참고링크를 보면 %28은 소괄호 ' ( '을 나타낸다.

URL Decode
URL Decode 사이트에서는 %28을 소괄호 ' ( '으로 잘 변환해준다.
그렇다는건 요나에서 괄호를 처리하는 로직부분이 잘못됐을것이라 추측이 된다.

@doortts
Copy link
Collaborator

doortts commented Aug 22, 2017

@E-nuri
가능하시다면 repo.yona.io 에 테스트 이슈 하나 만들어 주시겠어요?

@E-nuri
Copy link
Contributor Author

E-nuri commented Aug 22, 2017

@doortts 별도의 그룹을 만들어서 테스트 이슈를 남기면 될까요? 아니면 이슈를 따로 관리하는 그룹이 있을까요??

@benelog
Copy link

benelog commented Aug 23, 2017

marked 의 버그로 알고 있습니다.

markedjs/marked#109

@benelog
Copy link

benelog commented Aug 23, 2017

@sjstyle
Copy link
Collaborator

sjstyle commented Aug 23, 2017

RFC 1738( https://www.ietf.org/rfc/rfc1738.txt )에 보면 괄호가 URL에 포함될 수 있다고 되어있으므로 지금 동작이 오히려 맞는 것 같습니다.

RFC 1738 Page 3 일부 발췌

   Thus, only alphanumerics, the special characters "$-_.+!*'(),", and
   reserved characters used for their reserved purposes may be used
   unencoded within a URL.

@sjstyle
Copy link
Collaborator

sjstyle commented Aug 23, 2017

아 잘못 말하고 있었군요. 해당 동작이 패치되어야할 것으로 보이긴 한데 IE에서도 확인해보아야할 것 같습니다.

@E-nuri
Copy link
Contributor Author

E-nuri commented Aug 23, 2017

@sjstyle 퇴근 후 윈도우에서 테스트해보겠습니다. 윈10 크롬, 익스플로러, 엣지에서 테스트 후 결과 공유드리겠습니다.

@doortts doortts added this to the v1.8 milestone Sep 26, 2017
@doortts doortts removed this from the v1.8 milestone Oct 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants