simplesamlphp / simplesamlphp/simplesamlphp

adfs: enabling module breaks admin/federation page

オープン
#2,458 コメント 5 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

bug Confirmed medium
主要言語
PHP
スター
1.1k
フォーク
704
平均マージ
1日 15時間
マージ済み PR(30日)
4

説明

Help us to keep this issue-tracker clean! For questions or support, please refer to our mailing lists

Specifics of your environment

  1. IdP/SP/Proxy? IdP
  2. SimpleSAMLphp: 2.4.1
  3. PHP: What version are you using? 8.4.7
  4. Platform: unix or Windows? Unix
  5. Webserver: Apache/Nginx/ISS? Apache

Describe the bug
When accessing module.php/admin/federation an exception is thrown when the adfs-module is installed.

To Reproduce
Steps to reproduce the behavior:

  1. Install the latest version of the adfs-moduleGo to '...'
  2. module.php/admin/federation
  3. See error

Expected behavior
No exception

Screenshots or logs

Backtrace:
2 src/SimpleSAML/Error/ExceptionHandler.php:39 (SimpleSAML\Error\ExceptionHandler::customExceptionHandler)
1 vendor/symfony/error-handler/ErrorHandler.php:538 (Symfony\Component\ErrorHandler\ErrorHandler::handleException)
0 [builtin] (N/A)
Caused by: Symfony\Component\ErrorHandler\Error\UndefinedMethodError: Attempted to call an undefined method named "getHostedMetadata" of class "SimpleSAML\Module\adfs\IdP\ADFS".
Backtrace:
6 modules/admin/src/Controller/Federation.php:273 (SimpleSAML\Module\admin\Controller\Federation::getHostedIdP)
5 modules/admin/src/Controller/Federation.php:131 (SimpleSAML\Module\admin\Controller\Federation::main)
4 vendor/symfony/http-kernel/HttpKernel.php:181 (Symfony\Component\HttpKernel\HttpKernel::handleRaw)
3 vendor/symfony/http-kernel/HttpKernel.php:76 (Symfony\Component\HttpKernel\HttpKernel::handle)
2 vendor/symfony/http-kernel/Kernel.php:197 (Symfony\Component\HttpKernel\Kernel::handle)
1 src/SimpleSAML/Module.php:234 (SimpleSAML\Module::process)
0 public/module.php:17 (N/A)

Additional context
The solution is probably to implement the missing method in the adfs-module. If analysis supports this, this issue should be moved to the proper repository.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

modules/admin/src/Controller/Federation.php:273 から開始します。ここで getHostedIdP は ADFS クラスに存在しない getHostedMetadata メソッドを呼び出しています。インストール済みの adfs-module 実装とそのメタデータ処理を調査し、報告された環境で module.php/admin/federation を再現します。undefined-method exception が発生せずにページが読み込まれれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
php
領域
authentication
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。