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

컨텐츠 안에 mail to 링크가 있으면 에러가 발생합니다. #2174

Closed
wookho opened this Issue Nov 11, 2017 · 5 comments

Comments

3 participants
@wookho

wookho commented Nov 11, 2017

댓글이 950개 이상 달린 게시물을 읽을 경우 // 이건 아니고 제목의 문제로 확인되었습니다.
최고관리권한을 가진 계정에게 문제가 생깁니다.

콘솔에 아래 에러가 확인되며
xe.min.js?20171021181154:6 Uncaught Error: URNs do not have any generally defined hierarchical components

사용자정의 부분이 보이지 않고
스크립트 오류로 동작에 문제가 생깁니다.

댓글 페이지를 한페이지만 이전 페이지로 이동 시키면 이상이 없어집니다.

@wookho

This comment has been minimized.

Show comment
Hide comment
@wookho

wookho Nov 11, 2017

최고관리권한 -> 관리그룹 으로 정정합니다. 게시판 관리권한이 있는 계정으로 접근해도 동일합니다.

wookho commented Nov 11, 2017

최고관리권한 -> 관리그룹 으로 정정합니다. 게시판 관리권한이 있는 계정으로 접근해도 동일합니다.

@wookho

This comment has been minimized.

Show comment
Hide comment
@wookho

wookho Nov 11, 2017

이전 댓글을 삭제해서 910여개 정도 되니 정상이 됩니다.

wookho commented Nov 11, 2017

이전 댓글을 삭제해서 910여개 정도 되니 정상이 됩니다.

@wookho

This comment has been minimized.

Show comment
Hide comment
@wookho

wookho Nov 12, 2017

컨텐츠에 mail to 링크가 들어 있어 생기는 문제입니다.

wookho commented Nov 12, 2017

컨텐츠에 mail to 링크가 들어 있어 생기는 문제입니다.

@wookho wookho changed the title from 게시글에 댓글이 950개 이상 달릴 경우 문제가 발생합니다. to 컨텐츠 안에 mail to 링크가 있으면 에러가 발생합니다. Nov 12, 2017

@wookho

This comment has been minimized.

Show comment
Hide comment
@kijin

This comment has been minimized.

Show comment
Hide comment
@kijin

kijin Nov 12, 2017

Contributor

아래와 같이 해결하면 될 것 같습니다.

  • isSameHost() 함수에서 외부 링크 여부를 판단할 때 1) 프로토콜이 포함된 절대경로이면서 2) http: 또는 https:로 시작하지 않는 링크는 URL.js로 비교하지 않고 무조건 외부 링크로 취급하거나, 아예 target="_blank" 속성을 날려버림 (mailto: 또는 tel: 링크를 target="_blank"로 여는 것은 어차피 무의미합니다) (상대경로에 주의)
  • http: 또는 https:로 시작하는 링크라도 최소한의 형태가 제대로 갖춰져 있는지 확인한 후 URL.js로 넘겨주거나, 아니면 예외처리하는 코드를 작성
Contributor

kijin commented Nov 12, 2017

아래와 같이 해결하면 될 것 같습니다.

  • isSameHost() 함수에서 외부 링크 여부를 판단할 때 1) 프로토콜이 포함된 절대경로이면서 2) http: 또는 https:로 시작하지 않는 링크는 URL.js로 비교하지 않고 무조건 외부 링크로 취급하거나, 아예 target="_blank" 속성을 날려버림 (mailto: 또는 tel: 링크를 target="_blank"로 여는 것은 어차피 무의미합니다) (상대경로에 주의)
  • http: 또는 https:로 시작하는 링크라도 최소한의 형태가 제대로 갖춰져 있는지 확인한 후 URL.js로 넘겨주거나, 아니면 예외처리하는 코드를 작성

@bnu bnu self-assigned this Nov 13, 2017

@bnu bnu added this to the 1.8.47 milestone Nov 21, 2017

@bnu bnu added this to 처리 중 in 이슈 진행 상황 Nov 23, 2017

@bnu bnu modified the milestones: 1.8.47, 1.9.0 Nov 28, 2017

bnu added a commit that referenced this issue Nov 28, 2017

fix #2174 `mailto:` 등 URN 링크 문제 고침
- 페이지 내 링크가 다수 일 때 브라우저 성능 저하 문제 개선
- URI.js 업데이트. v1.19.0

@bnu bnu added the type/bug label Nov 28, 2017

@bnu bnu closed this Nov 29, 2017

이슈 진행 상황 automation moved this from 해결 중 to 이슈 종료 Nov 29, 2017

kijin added a commit to rhymix/rhymix that referenced this issue Nov 30, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment