Project

General

Profile

Backport(バックポート) #2070

OAuthのトークン発行時、Authorizationヘッダにパラメータを含めていた場合認証できない

Added by Masato Nagasawa over 11 years ago. Updated about 11 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2011-04-17
Due date:
% Done:

100%


Description

sfWebRequest 経由で oauth_token などの情報を取得していることが原因。


Related issues

Related to OpenPNE 3 - Bug(バグ) #2006: OAuthのトークン発行時、Authorizationヘッダにパラメータを含めていた場合認証できない Fixed(完了) 2011-04-17

Associated revisions

Revision fcce3fd1 (diff)
Added by Masato Nagasawa over 11 years ago

fixed has error if include in the 'Authorization' header (fixes #2070 BP from #2006)

Revision cf21bbe3 (diff)
Added by Masato Nagasawa over 11 years ago

Revert "fixed has error if include in the 'Authorization' header (fixes #2070 BP from #2006)"

This reverts commit fcce3fd1fed4dc4af7fca74b183dbf726aba28ae.

Revision ee540dff (diff)
Added by Masato Nagasawa about 11 years ago

fixed has error if include in the 'Authorization' header (fixes #2070, BP from #2006)

History

#1 Updated by Masato Nagasawa over 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 fcce3fd1fed4dc4af7fca74b183dbf726aba28ae で適用されました。

#2 Updated by Masato Nagasawa over 11 years ago

更新履歴 cf21bbe326db9d7e9ca9c0e40f3d845eb7641578 で適用されました。

#3 Updated by Masato Nagasawa over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to New(新規)
  • Assignee deleted (Masato Nagasawa)

3.4.12 では取り込みを行わないので、一旦取り消します。

#4 Updated by Shinichi Urabe over 11 years ago

  • Target version changed from OpenPNE 3.4.13 to OpenPNE 3.4.14

#5 Updated by Shinichi Urabe about 11 years ago

  • Target version changed from OpenPNE 3.4.14 to OpenPNE 3.4.15

親チケットが完了していないため修正延期

#6 Updated by Mutsumi Imamura about 11 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Mutsumi Imamura

#7 Updated by Masato Nagasawa about 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)

更新履歴 ee540dffdfe18d884a10c1088d2e35f28f4a842f で適用されました。

#8 Updated by Minoru Takai about 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューを行いました。

  • 3.4.x 向けの修正として適切か
    • 親チケットと差分が全く同じであることを確認しました。
    • 親チケット master ブランチと、 stable-3.4.x ブランチについて、本件の問題がバージョン間の差異に影響を受けるようには思わないので、 master と同一の修正を取り込むことは妥当だと判断しました。
      • 万一、本件に関連した不具合が stable-3.4.x ブランチに存在した場合は、それが発覚した時点で修正を行うことを予定します。現時点ではそのような不具合が予見できていないので、現時点の修正内容で OK と判断します。
  • 問題を解決するための修正として適切か
    • これについては takai では判断できていませんが、親チケットでレビューが通っていることを以て OK と判断しています。

#9 Updated by Mutsumi Imamura about 11 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テスターテストが難しいため、レビューのみで修正OKとします。

Also available in: Atom PDF