Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[转 #5135] PCL 不会自动读取 X-Authlib-Injector-API-Location 标头 #5168

Open
1 task done
shimoranla opened this issue Nov 23, 2024 · 16 comments
Open
1 task done
Labels
· Bug 处理中 开发者正在调查或处理该项 🟩 低 优先度:低

Comments

@shimoranla
Copy link

shimoranla commented Nov 23, 2024

检查项

描述

根据 规范
皮肤站会在启动器请求时返回一个 X-Authlib-Injector-API-Location 标头来帮助启动器确认 Yggdrasil 认证服务器的实际地址
但是 PCL 实际上并不会读取这个标头,导致无法登录部分站内文档直接提供的地址实际路径和规范不符合的皮肤站
Like #5135

重现步骤

找一个地址不规范但提供标头的皮肤站 (ely.by)
填写认证地址,登录即可复现

日志与附件

Log1.txt

这个其实可以自行复现.jpg

@shimoranla shimoranla added · Bug 新提交 需要社区初步确认其有效的新提交 labels Nov 23, 2024
@Pigeon0v0 Pigeon0v0 added 等待确认 已经过社区确认,等待开发者确认 and removed 新提交 需要社区初步确认其有效的新提交 labels Nov 23, 2024
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟨 中 优先度:中 and removed 等待确认 已经过社区确认,等待开发者确认 labels Nov 23, 2024
@LTCatt LTCatt changed the title [Bug] PCL 不会自动读取 X-Authlib-Injector-API-Location 标头 [转 #5135] PCL 不会自动读取 X-Authlib-Injector-API-Location 标头 Nov 23, 2024
@allMagicNB
Copy link
Contributor

allMagicNB commented Nov 24, 2024

谁地址不规范了?

POST https://account.ely.by/api/authlib-injector/authserver/authenticate

{
  "username": "********"
  "password": "********"
}

@shimoranla
Copy link
Author

shimoranla commented Nov 24, 2024

谁地址不规范了?

文档内提供的地址是 https://authserver.ely.by/auth/authenticate
我认为提供 ALI 是出于兼容性考虑
如果你认为我说的不对,你可以开一个新的 Issue
btw:我不想和你吵,所以请放缓你的语气

@allMagicNB
Copy link
Contributor

谁地址不规范了?

文档内提供的地址是 https://authserver.ely.by/auth/authenticate
我认为提供 AIL 是出于兼容性考虑
如果你认为我说的不对,你可以开一个新的 Issue
btw:我不想和你吵,所以请放缓你的语气

文档中也提供了如何使用 authlib-injector。

@shimoranla
Copy link
Author

shimoranla commented Nov 24, 2024

文档中也提供了如何使用 authlib-injector。

文档内提供的地址为 ely.by,经查证,对应的服务器也提供了 ALI 标头

图片

image

@allMagicNB
Copy link
Contributor

authlib-injector 会自动将参数 ALI 到正确网址,同时启动器也应当实现。
我在说的问题是:Ely.by 有自己的验证系统的同时,也能使用 authlib-injector。
但是 ALI 到的正确地址并非不规范。

@shimoranla
Copy link
Author

我在说的问题是:Ely.by 有自己的验证系统的同时,也能使用 authlib-injector。

没太懂什么意思,如果是想要 PCL 支持站内的 OAuth 登录系统... 你可以去问问 @wuliaodexiaoluo 能不能做(或者找鸽秋.jpg)
如果你认为这个 Issue 没有涉及到你的实际需求,可以开一个新的
同时我还能拿一个活跃橙(虽然没快照版也用不了就是了)

但是 ALI 到的正确地址并非不规范。

这算表述有误,已经改了,不过我的原意是指 ely.by 文档直接提供的路径和规范要求的不符

@allMagicNB
Copy link
Contributor

allMagicNB commented Nov 24, 2024

要不你先自己去试试将认证服务器填为 https://account.ely.by/api/authlib-injector 后登录。
并且此为 PCL 不遵守规范导致,并非 Ely.by 的问题。

@wuliaodexiaoluo

This comment was marked as resolved.

@allMagicNB
Copy link
Contributor

……你看的文档那是人家自己的规范。

@wuliaodexiaoluo
Copy link
Contributor

……你看的文档那是人家自己的规范。

显然 PCL 并没有考虑到会有通过 ALI 提供验证服务器地址的皮肤站,不过如果只是解决无法登陆 Ely.by,那么这个 Issue 已经覆盖了这个问题(((

@LTCatt
Copy link
Member

LTCatt commented Dec 5, 2024

PCL 使用 https://account.ely.by/api/authlib-injector 可以正常登录。
我没找到这个网站在哪里提供了使用 ALI 的 API 地址?谁可以发一下么?

@LTCatt LTCatt added 等待提交者 需要提交者补充信息或进行操作 and removed 处理中 开发者正在调查或处理该项 🟨 中 优先度:中 labels Dec 5, 2024
@shimoranla
Copy link
Author

shimoranla commented Dec 5, 2024

我没找到这个网站在哪里提供了使用 ALI 的 API 地址?谁可以发一下么?

ely.by 官网有提供

image

Edit: 对于 authserver.ely.by 和 account.ely.by,提供的 ALI 为相对地址

@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟨 中 优先度:中 🟩 低 优先度:低 and removed 等待提交者 需要提交者补充信息或进行操作 🟨 中 优先度:中 labels Dec 6, 2024
@LTCatt
Copy link
Member

LTCatt commented Dec 6, 2024

https://account.ely.by/ 能找到这个标头,不过我这里 https://ely.by/ 还是没有。
image

@shimoranla
Copy link
Author

龙猫原来在用 360 吗(震撼)

我觉得大概率是 360 的问题,不过因为我不用 360 也不能肯定就是了(

image

@allMagicNB
Copy link
Contributor

allMagicNB commented Dec 6, 2024

https://account.ely.by/ 能找到这个标头,不过我这里 https://ely.by/ 还是没有。
image

curl -I "https://ely.by"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· Bug 处理中 开发者正在调查或处理该项 🟩 低 优先度:低
Projects
None yet
Development

No branches or pull requests

5 participants