Backport（バックポート） #3684: opSecurityUser::login() で生成される member/setSid アクションへリダイレクトするために渡される個々のパラメータがURLエンコードされていない - OpenPNE 3 - OpenPNE Issue Tracking System

Actions

Copy link

Backport（バックポート） #3684

closed

opSecurityUser::login() で生成される member/setSid アクションへリダイレクトするために渡される個々のパラメータがURLエンコードされていない

Added by 誠二天重 over 10 years ago. Updated over 10 years ago.

Status:

Fixed（完了）

Priority:

Normal（通常）

Assignee:

誠二天重

Target version:

OpenPNE 3.8.13

Start date:

2012-10-01

Due date:

% Done:

100%

Estimated time:

Description

現象¶

opSecurityUser::login() 内に以下のような処理があり、パラメータをurlencode()せず、文字結合している。
この $uri の返値は、現時点では opMemberAction::login() でのみ使われており (本チケットはOpenPNE本体のみのソースを追って確認したもので、プラグインなどで別の用途でこの返値を使っている場合、このチケットの修正目的からすると影響を受けてしまう。)、ここからリダイレクト用のURLとして扱われているので、urlencode() されている必要のあるパラメータだと考えられる。

is_remember_login のパラメータについてはintでキャストしているため問題ないと考えられる
next_uri, sid, ts のパラメータについては、URLとして意味を持つ文字列が含まれる可能性があるため、urlencode() が必要と考えられる

        $uri = '@member_setSid?next_uri='.$uri
             .'&is_remember_login='.(int)$this->getAuthAdapter()->getAuthForm()->getValue('is_remember_me')
             .'&sid='.$item[0]
             .'&ts='.$item[1];