9001 / 9001/copyparty

idp-logout option doesnt properly substitute redirection destination

未关闭 适合新手
#954 3 条评论 0 个 reaction 已指派 1 人 已被 @9001 认领 在 GitHub 查看
bug
主要语言
Python
星标
46.7k
派生
1.9k
平均合并
9 天 2 小时
30 天内合并 PR
4

描述

### Describe the bug
idp-logout does not replace {dst} with the redirect destination, and it is instead left in the url. idp-login works right.

### To Reproduce
Use the idp-logout option with a url including {dst}

### Expected behavior
{dst} replaced with the proper redirect

### Server details
* **how you're running copyparty:** docker
* **docker image:** copyparty/ac, v1.19.19

贡献指南

打开贡献指南

调研方向

在 copyparty 源代码中查找处理 idp-logout 的代码,可能位于与身份验证或 URL 路由相关的文件中。将其与 idp-login 的实现进行比较,以了解其中如何进行 {dst} 替换。通过使用包含 {dst} 的 idp-logout URL 运行 copyparty 来测试修复,并验证重定向目标是否正确替换了占位符。

由索引模型根据 Issue 内容生成。

评估

技术栈
python
领域
backend
Issue 类型
缺陷
难度
2/5
预计耗时
1-3 小时
活跃度
停滞
描述清晰度
描述清楚
新手友好度
65/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。