[WIP]支払い方法を変更した時に支払い方法のアンカーへ移動

[tao-s]

概要(Overview・Refs Issue)

支払い方法を変更した場合にページトップではなく支払い方法のアンカーへ

#5186

テスト（Test)

稼働しているコードですが、ShoppingController::redirectTo()を変えてるので他の環境だとエラーが出る可能性があるので確認してください。

相談（Discussion）

できればControllerのannotationでflagmentの設定したいけどよくわからん。

tao-s@0f69bcd#diff-21c4d1297a9e1961c312211f32d8c1184a995d79a993bc53535638dfceda5049R208
この辺りもっと良いやり方がありそう。あとこの書き方だとバグが出そう

$redirectTo = explode("#",$redirectTo);
$fragment = $redirectTo[1];
$redirectTo = $redirectTo[0];

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[nanasess]

大丈夫かと思いますが、 URL fragment を動的に追加している箇所は念には念を入れてペネトレーションテストも追加しておきたいです
refs #5169

[xuelian311]

@tao-s こちらの件ご対応いただけそうでしょうか？ぜひ次のバージョンで入れさせていただきたいです！

[tao-s]

@xuelian311 遅くなりすいません。これって何を対応すれば良いんですか？

[tao-s]

@nanasess これってテスト書けってことなんですかね？

[nanasess]

@tao-s url-fragment 攻撃の問題ないことを、OWASP Zap のテスト書いて確認したかったのですが、テストの構成が変わって難しくなってしまいましたね。。。

手動でテストするしかないか🤔

[ji-eunsoo]

@nanasess @tao-s
ありがとうございます！
こちらですが、テストの追加はできないという認識で大丈夫でしょうか？

[nanasess]

@ji-eunsoo 手動でテストするしかないと思います