Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
継承表示されているフレームの設定画面を子ページ側から開こうとすると権限エラー(403)になる問題を修正しました。
変更内容
ConnectPageのpage_id/frame_id整合性判定を見直し、メインエリアは従来どおり完全一致、共通エリアは祖先ツリー上で実際に採用される継承元ページのフレームのみ許可するようにしました。ConnectPageFrameValidationFeatureTestを追加し、継承許可・拒否の境界条件をhandle()の公開振る舞いとして確認できるようにしました。背景と目的
メイン以外のフレームは親ページから子ページへ継承表示されますが、
ConnectPageの厳密な一致判定により、子ページに表示されている継承フレームでも設定画面を開こうとすると 403 になることがありました。今回の修正では、実際に画面上で有効な継承フレームだけを許可しつつ、別系統ページやメインエリアの不正な組み合わせは引き続き拒否できるようにしています。
特記事項
ConnectPageの共通判定のため、同じ判定を使うフレーム関連ルート全体に反映されます。ConnectPage::handle()の公開振る舞いで確認しています。Router/Routeを利用しています。レビュー完了希望日
軽微な不具合修正のため、お手すきの際にご確認をお願いいたします。
関連Pull requests/Issues
なし
DB変更の有無
無し
チェックリスト