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

Experimental: handle redirect from slashed URL #1265

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

haxibami
Copy link
Member

@haxibami haxibami commented Nov 11, 2024

問題

  • Astroでは、build.formatpreserve に設定されている(現状)場合、/foo/(スラッシュあり)からのリダイレクトも /foo(スラッシュなし)からのリダイレクトと同じように扱われてしまう(前者では dist/foo/index.html を出力してほしいところ、dist/foo.html が出力される)
  • さらにGitHub Pagesでは、スラッシュありのURL(/foo/)でアクセスするとスラッシュなしのURL(/foo)に転送されない
  • その結果、/foo/ からのリダイレクトが一切設定できなくなってしまう

修正

  • /foo/ からのリダイレクトを /foo/index からのリダイレクトとみなし、強引に dist/foo/index.html を生成させる
  • 細かな問題点:dist/foo/index.html の文面に /foo/index という文字列が含まれてしまう

備考

  • build.formatdirectory とすればそもそも問題は生じない(すべてのページが index.html で生成される)ため、このようなハックは不要になる

Copy link

🚀 Deployed on https://deploy-preview-1265--utelecon.netlify.app

@haxibami haxibami added the web-development Tasks of web-development category label Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
web-development Tasks of web-development category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant