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

修改 Referer 的字段顺序以符合正常语义 #5523

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Silverteal
Copy link
Contributor


http://{version}.pcl2.open.server/
修改为
http://{version}.open.pcl2.server/

@3gf8jv4dv 3gf8jv4dv added · 优化 等待确认 已经过社区确认,等待开发者确认 labels Jan 26, 2025
@LTCatt
Copy link
Member

LTCatt commented Jan 26, 2025

umm 我确实理解这样改完更好,但现在作为某种意义上的公开 API 的一部分,去做笔误类的修改,导致更多人需要更新 UA 检查,就有点得不偿失了……
所以,暂时就让它这样吧 Orz

@LTCatt LTCatt closed this Jan 26, 2025
@LTCatt LTCatt added 暂无计划 短期内没有制作该项的计划 and removed 等待确认 已经过社区确认,等待开发者确认 labels Jan 26, 2025
@Silverteal Silverteal deleted the patch-1 branch January 27, 2025 01:29
@Silverteal Silverteal restored the patch-1 branch January 27, 2025 01:29
@Silverteal
Copy link
Contributor Author

我的UA检查是*.pcl2.server,事实上修改前的才需要特判(

@LTCatt
Copy link
Member

LTCatt commented Jan 27, 2025

……那也可以 orz
@Light-Beacon 看看你那里在合并后会不会炸?

@LTCatt LTCatt reopened this Jan 27, 2025
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟨 中 优先度:中 and removed 暂无计划 短期内没有制作该项的计划 labels Jan 27, 2025
@Light-Beacon
Copy link
Collaborator

……那也可以 orz @Light-Beacon 看看你那里在合并后会不会炸?

我的没问题 也是 *.pcl.server

@z0z0r4
Copy link

z0z0r4 commented Jan 29, 2025

很久以前就想知道这 referer 是谁写的了(

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

Successfully merging this pull request may close these issues.

5 participants