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

CNAME 自定义域名似乎不支持 #18

Open
StringKe opened this issue Jun 24, 2024 · 11 comments · Fixed by #19
Open

CNAME 自定义域名似乎不支持 #18

StringKe opened this issue Jun 24, 2024 · 11 comments · Fixed by #19
Assignees
Labels
enhancement New feature or request

Comments

@StringKe
Copy link

StringKe commented Jun 24, 2024

cname: true,

查到有这个配置,查代码层没找到代码 cname 的处理

@fengmk2

@fengmk2
Copy link
Member

fengmk2 commented Jun 24, 2024

@StringKe cname 的用法我没印象了,有具体的文档么?

@StringKe
Copy link
Author

没有什么文档,就是自定义域名之后所有的访问地址都是自定义的地址,不需要携带任何 region 了。

官网上都是关于如何自定义域名相关配置,使用上就是前面的 bucket.region.xxx.com 这个变成了绑定的自定义域名

@fengmk2
Copy link
Member

fengmk2 commented Jun 25, 2024

@StringKe 好的,我看看。

@fengmk2
Copy link
Member

fengmk2 commented Jun 25, 2024

大概明白了,我实现一下。

@fengmk2
Copy link
Member

fengmk2 commented Jun 25, 2024

@StringKe #19

@fengmk2 fengmk2 self-assigned this Jun 25, 2024
@fengmk2 fengmk2 added the enhancement New feature or request label Jun 25, 2024
@StringKe
Copy link
Author

StringKe commented Jun 26, 2024

@fengmk2 这个实现似乎不正确,签名没有 bucket 参与了

CleanShot 2024-06-26 at 12 04 01@2x

CleanShot 2024-06-26 at 12 06 34@2x

@fengmk2
Copy link
Member

fengmk2 commented Jun 26, 2024

看来 bucket 还是得必传。

@fengmk2 fengmk2 reopened this Jun 26, 2024
@fengmk2
Copy link
Member

fengmk2 commented Jun 26, 2024

晚点我先自己验证完成再提交 bugfix 。

@fengmk2
Copy link
Member

fengmk2 commented Jun 26, 2024

@StringKe 或者你来尝试修复一下?

@StringKe
Copy link
Author

@fengmk2 我不太清楚那些地方需要 bucket 参与。

我目前的方案是返回 url 的时候直接暴力 replace。不太适合作为一种方案。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants