simplesamlphp / simplesamlphp/simplesamlphp
adfs: enabling module breaks admin/federation page
还没有人认领这个 Issue。
- 主要语言
- 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
- IdP/SP/Proxy? IdP
- SimpleSAMLphp: 2.4.1
- PHP: What version are you using? 8.4.7
- Platform: unix or Windows? Unix
- 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:
- Install the latest version of the adfs-moduleGo to '...'
- module.php/admin/federation
- 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.
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 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