simplesamlphp / simplesamlphp/simplesamlphp

adfs: enabling module breaks admin/federation page

未关闭
#2,458 5 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

bug Confirmed medium
主要语言
PHP
星标
1.1k
派生
704
平均合并
1 天 15 小时
30 天内合并 PR
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. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 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 摘要。