Enhancement（機能追加・改善） #2341
完了
セッションキーをURLパラメータで引き回すことを強制する設定の追加
Shingo Yamada さんが13年以上前に追加.
ほぼ13年前に更新.
説明
Overview (概要)¶
cookieの使用が前提のキャリア・端末で、何らかの原因で cookie が使用できない場合、現状のOpenPNEではログインできなくなる。
その救済措置として、セッションキーをURLパラメータで引き回すことを強制する設定を追加する。
Spec (仕様)¶
セッションキーをURLパラメータで引き回すことを強制する設定を追加する
Way to Fix（修正方法）¶
携帯でcookieを用いるかどうかの設定をOpenPNE.ymlに記述し、cookieを用いることができる端末かどうかを調べる際に設定値を確認する実装を行った。
- 対象バージョン を OpenPNE3.6beta13 から OpenPNE 3.7.0 に変更
- 担当者 を Kousuke Ebihara から Yuya Watanabe に変更
- 優先度 を Normal（通常） から High（高め） に変更
- ステータス を New（新規） から Accepted（着手） に変更
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
- config/OpenPNE.yml.sample の is_use_mobile_cookie の設定ですが、コメントインしてください。この状態ではデフォルト値が false である（Cookie を使いたい場合はコメントアウトして使う）と誤解されかねません。ここでの設定値は真偽値なのですから、 is_use_mobile_cookie: true をコメントなしで設定させておいて、必要が生じた場合に is_use_mobile_cookie: false に変更させるようになっていたほうが直感的でわかりやすいです。
- もともとあった if 文の端末判定のための条件式のなかに $isUseMobileCookie 変数のチェックを追加していますが、これはあまり直感的ではないと思います。もともとの条件式はまあ複雑ではあるといえ端末判定であるということがある程度明確でしたが、ここに $isUseMobileCookie の判定が加わったことでその意味が失われました。このメソッドにおいて、 $isUseMobileCookie が false なのであれば、端末判定などする必要もなく false を返却して構わないので、それを示すために、もともとあった端末判定の if 文の前、 op_is_use_mobile_cookie の設定値を取得した直後のタイミングで別の if 文として $isUseMobileCookie の判定をおこなうことで、 $isUseMobileCookie の判定と端末の判定それぞれに別々の意図が生まれ、可読性が向上すると思いますので修正を検討してください。
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
テスト完了致しました。
問題ありませんのでステータスをFixedに致します。
他の形式にエクスポート: Atom
PDF
