ユーザー API キー: ペイロードと既存のクエリ文字列がダブルクエスチョンマークを生成する

blattersturm · 2019 年 7 月 22 日午前 10:55

auth_redirect にすでにクエリ文字列が含まれている場合（例：http://localhost:30120/auth-discourse?state=BASE64）、user-api-key/new を呼び出すと、http://localhost:30120/auth-discourse?state=BASE64?payload=PAYLOAD のようなリダイレクト URI が生成されてしまいます。クエリ文字列が 2 つあり、& で連結されていません。

github.com/discourse/discourse

app/controllers/user_api_keys_controller.rb

888e68a16


      
          
          public_key = OpenSSL::PKey::RSA.new(params[:public_key])
          @payload = Base64.encode64(public_key.public_encrypt(@payload))
          
          if scopes.include?("one_time_password")
            # encrypt one_time_password separately to bypass 128 chars encryption limit
            otp_payload = one_time_password(public_key, current_user.username)
          end
          
          if params[:auth_redirect]
            redirect_path = +"#{params[:auth_redirect]}?payload=#{CGI.escape(@payload)}"
            redirect_path << "&oneTimePassword=#{CGI.escape(otp_payload)}" if scopes.include?("one_time_password")
            redirect_to(redirect_path)
          else
            respond_to do |format|
              format.html { render :show }
              format.json do
                instructions = I18n.t("user_api_key.instructions", application_name: @application_name)
                render json: { payload: @payload, instructions: instructions }
              end
            end

このコードは、URI にすでにクエリ文字列が含まれているかを確認するように変更する必要があるか、あるいは URI ビルダーを使用する必要があるかもしれません。

eviltrout · 2019 年 7 月 22 日午後 1:41

コードが前提としているのは、クエリパラメータがまだ含まれていないという点のようです。この問題を修正するプルリクエスト（PR）の提出を歓迎します。

saurabhp · 2019 年 7 月 23 日午後 3:50

この問題に対する PR を以下の場所に作成しました:

saurabhp · 2019 年 7 月 24 日午前 2:37

PR がマージされました。このトピックはクローズできます

トピック		返信	表示
Feasibility of allowing a User Api Key client to register a valid auth_redirect Development	7	376	2024 年 11 月 11 日
Redirect after login loses query params Development	9	2133	2019 年 7 月 8 日
Support wildcard in "allowed user api auth redirects" setting Feature	0	646	2018 年 12 月 15 日
Per User API Keys Not Working Support	2	1208	2021 年 9 月 2 日
Generating User Api Keys with REST API Development rest-api	20	8714	2018 年 4 月 8 日

ユーザー API キー: ペイロードと既存のクエリ文字列がダブルクエスチョンマークを生成する

関連トピック