테스트 사이트 - 개발 중인 베타 버전입니다

로그인 부분 이렇게 사용해도 문제없을까요? 채택완료

시린♡ 4년 전 조회 1,774

그누 패치중에 기존에 사용하던게 로그인페이지 유지 적용해놨는데

url 체크부분에 원본은

if ($url) {
    // url 체크
    check_url_host($url, '', G5_URL, true);

    $link = urldecode($url);

 

이렇게 되어있고 수정은

 

if($urls[1]){
    // url 체크
    check_url_host($url[1]);
 
    $link = urldecode($urls[1]);     

이렇게 사용중인데 큰문제 없겠죠? '', G5_URL, true 이부분이 추가된거같은데

만약에 필요없다면 그냥 사용해도 될꺼같은데 혹시라도 필요하다면 

어떤식으로 check_url_host($url[1]); 여기에 대입시켜줘야할까요?

댓글을 작성하려면 로그인이 필요합니다.

답변 1개

채택된 답변
+20 포인트
검은냥냥이

common.lib.php

G:3273 쪽에 보시면 특정 버젼에 의한 parse_url 버그 때문에 그런것 같네요.

관련 내용 확인하시고 아니면 굳이 맞출 필요는 없습니다.

 

</p>

<p>    //php 5.6.29 이하 버전에서는 parse_url 버그가 존재함</p>

<p>    //php 7.0.1 ~ 7.0.5 버전에서는 parse_url 버그가 존재함</p>

<p>    if ( $is_redirect && (isset($p['host']) && $p['host']) ) {</p>

<p>        $bool_ch = false;</p>

<p>        foreach( array('user','host') as $key) {</p>

<p>            if ( isset( $p[ $key ] ) && strpbrk( $p[ $key ], ':/?#@' ) ) {</p>

<p>                $bool_ch = true;</p>

<p>            }</p>

<p>        }</p>

<p>        if( $bool_ch ){</p>

<p>            $regex = '/https?\:\/\/'.$host.'/i';</p>

<p>            if( ! preg_match($regex, $url) ){</p>

<p>                $is_host_check = true;</p>

<p>            }</p>

<p>        }</p>

<p>    }</p>

<p>

로그인 후 평가할 수 있습니다

답변에 대한 댓글 1개

시린♡
4년 전
아하..그러면 그냥 사용해도 큰문제는 없을꺼같네여...
감사합니다~~

댓글을 작성하려면 로그인이 필요합니다.

답변을 작성하려면 로그인이 필요합니다.

로그인