halo icon indicating copy to clipboard operation
halo copied to clipboard

/login 可能会被页面的别名覆盖掉

Open ruibaby opened this issue 1 year ago • 2 comments

系统信息

使用的哪种方式运行?

Docker

发生了什么?

测试发现,Halo 的页面功能可以覆盖掉 /login 路由,可能会导致无法再正常访问 /login 页面。建议禁止系统保留的路由名称,或者支持安全模式(开启之后不允许主题覆盖 /login,不允许页面路由覆盖 /login)。

/kind improvement /area core

复现步骤

No response

相关日志输出

No response

附加信息

No response

ruibaby avatar Oct 18 '24 06:10 ruibaby

可以尝试将下面这行中的 login 修改为 plugin:system:login

https://github.com/halo-dev/halo/blob/c465bf8c75266af5e77e368c4ddb87e2e231516d/application/src/main/java/run/halo/app/security/preauth/PreAuthLoginEndpoint.java#L102

JohnNiang avatar Oct 18 '24 06:10 JohnNiang

可以尝试将下面这行中的 login 修改为 plugin:system:login

halo/application/src/main/java/run/halo/app/security/preauth/PreAuthLoginEndpoint.java

Line 102 in c465bf8

ServerResponse.ok().render("login", Map.of(

这个方法不行,这是页面的路由优先级高导致轮不到这里

guqing avatar Oct 18 '24 09:10 guqing

/assign

guqing avatar Oct 22 '24 10:10 guqing