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

한글도메인 사용시 자바스크립트 isSameHost() 함수가 오작동하는 것을 고침 #2264

Merged
merged 1 commit into from Jun 4, 2018

Conversation

2 participants
@kijin
Contributor

kijin commented Jun 1, 2018

한글도메인 사용시 동일한 도메인을 두 가지 이상의 형태로 표현할 수 있습니다. 이것 때문에 isSameHost() 함수가 동일한 도메인 여부를 제대로 판단하지 못합니다. URI.js에서 제공하는 normalizeHostname() 함수를 사용하여 punycode로 통일한 후 비교하도록 수정합니다.

불필요하게 포트가 붙은 주소도 정확하게 비교하도록 하기 위해 normalizePort() 함수도 추가합니다.

사실 URI.js에서 제공하는 normalize() 함수 하나만 호출하면 모두 정규화시켜 주지만, 쿼리스트링이나 fragment 등 불필요한 부분을 처리하느라 자원이 낭비되므로 꼭 필요한 3가지(hostname, port, pathname)만 정규화하도록 했습니다.

참고: rhymix/rhymix#1037 thanks to @misol

@bnu bnu self-assigned this Jun 4, 2018

@bnu bnu added this to the 1.9.7 milestone Jun 4, 2018

@bnu bnu merged commit dbc3954 into xpressengine:develop Jun 4, 2018

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details

@bnu bnu added this to 이슈 종료 in 이슈 진행 상황 via automation Jun 4, 2018

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