Skip to content

[R-2025-23]多要素認証機能の本番環境への設定作業(rebase+テストコードの追加)#717

Open
Yusaku-Kitabatake wants to merge 53 commits into
developfrom
feature/R-2025-23-56420-add-tests
Open

[R-2025-23]多要素認証機能の本番環境への設定作業(rebase+テストコードの追加)#717
Yusaku-Kitabatake wants to merge 53 commits into
developfrom
feature/R-2025-23-56420-add-tests

Conversation

@Yusaku-Kitabatake

@Yusaku-Kitabatake Yusaku-Kitabatake commented Apr 7, 2026

Copy link
Copy Markdown

Purpose

https://redmine.devops.rcos.nii.ac.jp/issues/55791

Changes

  • models.UserにAALおよびIAL属性を追加
    • IdP経由のサインイン時、IdPから送信されたSAMLレスポンスに含まれるeduPersonAssurance属性からAALおよびIALの値を取得しmodels.Userに格納する
    • eduPersonAssurance属性にAALの値が存在しない場合、Shib-AuthnContext-Classパラメータを参照する
  • LoA機能(models.loa)の実装
    • IdP経由のサインイン時、IdP側に要求するAALおよびIALの値の設定(要求値を満たさない場合サインイン不可)
    • IdPにAAL2相当の認証を要求するための導線(MFA URL)の定義およびUI(MFAボタン)の表示制御
  • テストコードの追加
    • eduPersonAssurance から IAL/AAL を判定する処理
    • LoA テーブルを参照して要件を満たすか検証する処理
    • 要件未達時に ValidationError を raise する処理
    • MFA URL を urlencode で組み立てて user.context に格納する処理
    • user.ial / user.aal を DB に永続化する処理
    • idp_attrinstitution.id を含めて保存する処理

Dev-Ops Notes

本機能は
RCOSDP/RDM-cas-overlay#19
の改修と連動しているため、本機能をリリースする際は上記のPR内commitも合わせてリリースすること。

また、実行環境の設定については
https://redmine.devops.rcos.nii.ac.jp/issues/55790
こちらのcommitを反映させること。

QA Notes

Documentation

Side Effects

Ticket

https://redmine.devops.rcos.nii.ac.jp/issues/56420
https://redmine.devops.rcos.nii.ac.jp/issues/58907

tma-hcphat and others added 20 commits December 31, 2025 10:23
…i_grdm_202510_step2/2.3_mapcore_group_management_rebase
@Yusaku-Kitabatake Yusaku-Kitabatake force-pushed the feature/R-2025-23-56420-add-tests branch from 06bbe40 to a680a10 Compare April 8, 2026 03:20
@Yusaku-Kitabatake Yusaku-Kitabatake force-pushed the feature/R-2025-23-56420-add-tests branch from a680a10 to 6897f8c Compare April 8, 2026 05:57
@Yusaku-Kitabatake Yusaku-Kitabatake marked this pull request as draft April 8, 2026 07:44
@Yusaku-Kitabatake Yusaku-Kitabatake marked this pull request as ready for review April 9, 2026 02:23
@Yusaku-Kitabatake Yusaku-Kitabatake marked this pull request as draft April 9, 2026 02:26
@Yusaku-Kitabatake Yusaku-Kitabatake marked this pull request as ready for review April 13, 2026 05:15
tma-hcphat and others added 28 commits May 22, 2026 16:36
- Isolate or reset the `isComposing` flag to prevent pollution between modals.
- Explicitly initialize the `isComposing` property in the controller.
- Update UT test to ensure tests accurately verify the source code behavior.
…i_grdm_202603_step9/2.2_improve_project_name_input
…i_grdm_202603_step9/2.1_idp_auth_response_text_display
…i_grdm_202510_step2/2.3_mapcore_group_management_rebase and resolve conflict
…_mapcore_group_management_rebase

Feature/nii grdm 202510 step2/2.3 mapcore group management rebase
…_improve_project_name_input

Feature/nii grdm 202603 step9/2.2 improve project name input
…ights-migration

Fix/remove metadata access rights migration
Fixed bug in creating ro-crate without additional metadata files.
…_grdm_202510_step2/2.1_improve_multiple_small_file_uploads_merge and resolve conflict
 + Add PROVIDER_SETTINGS for s3compatsigv4 and Institution storage
 + Add reset default handle logic
…improve_multiple_small_file_uploads_merge

Hotfix/nii grdm 202510 step2/2.1 improve multiple small file uploads merge
…_idp_auth_response_text_display

Feature/nii grdm 202603 step9/2.1 idp auth response text display
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants