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.

Sign up for GitHub

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

Jump to bottom

[BUG] 网站管理,网站域名多端口情况,导航链接错误,应该是域名端口访问形式. #4964

Open
AtlantisDe opened this issue May 10, 2024 · 6 comments
Open

[BUG] 网站管理,网站域名多端口情况,导航链接错误,应该是域名端口访问形式. #4964

AtlantisDe opened this issue May 10, 2024 · 6 comments
Assignees
@wanghe-fit2cloud
Labels
类型:优化
Milestone
v1.10.10-lts

Comments

@AtlantisDe
Copy link

联系方式

No response

1Panel 版本

v1.10.7-lts

问题描述

[BUG] 网站管理,网站域名多端口情况,导航链接错误,应该是域名端口访问形式.

image

重现步骤

期待的正确结果

域名:端口
IP:端口
如果有ssl 应该加上判断.

相关日志输出

No response

附加信息

No response
@wanghe-fit2cloud
Copy link
Member

提供一下 1Panel 版本。

在开发测试环境未能复现上述问题。

@AtlantisDe
Copy link
Author

提供一下 1Panel 版本。

在开发测试环境未能复现上述问题。

上面不写了吗? 老版本 v1.10.7-lts 新版 都存在这个问题.

@zhengkunwang223
Copy link
Collaborator

您好 经过验证 是 https 模式下跳转有问题
后续版本优化

@zhengkunwang223 zhengkunwang223 added this to the v1.10.8-lts milestone May 11, 2024
@zhengkunwang223
Copy link
Collaborator

由于我们暂时还没有标记添加的 域名：端口 中是否为 https
所以跳转的时候 只能跳转到 http:域名:端口

@zhengkunwang223 zhengkunwang223 mentioned this issue May 13, 2024
Merged
f2c-ci-robot bot pushed a commit that referenced this issue May 13, 2024
@zhengkunwang223
feat: 优化网站域名跳转 (#4998)
5a8869e 
Refs #4964
ssongliu pushed a commit that referenced this issue May 13, 2024
@zhengkunwang223 @ssongliu
feat: 优化网站域名跳转 (#4998)
bf60e99 
Refs #4964
@AtlantisDe
Copy link
Author

等你们发布新版了我验证下 是否修复了 我在关闭这个 issue

@AtlantisDe
Copy link
Author

已经 检查还是有小问题. 如果能修下更好了.
https 判断下 端口如果包含443 默认就不要添加端口
否者 https 是其它端口 还是 保持 https://:端口 形式 进行导航.
#4998

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wanghe-fit2cloud wanghe-fit2cloud

Labels
类型:优化
Projects
None yet
Milestone
v1.10.10-lts
Development

No branches or pull requests
3 participants
@zhengkunwang223 @AtlantisDe @wanghe-fit2cloud