Skip to content

Keep the calyptia-fluentd package browser links working by disabling Netlify's Pretty URLs - #521

Merged
kenhys merged 1 commit into
fluent:masterfrom
Watson1978:disable-netlify-pretty-urls
Oct 8, 2026
Merged

kenhys merged 1 commit into
fluent:masterfrom
Watson1978:disable-netlify-pretty-urls

Conversation

@Watson1978

Copy link
Copy Markdown
Contributor

On www.fluentd.org, links whose path is /index.html lose index.html even when they point to other hosts. The calyptia-fluentd package browser links on /download/calyptia_fluentd (https://calyptia-fluentd.s3.us-east-2.amazonaws.com/index.html?prefix=1) become /?prefix=1 and open an XML listing instead. The source in source/download/calyptia_fluentd.html.erb is correct, so the rewrite comes from Netlify's Pretty URLs post processing.

This sets pretty_urls = false in netlify.toml (Netlify docs). The same setting stopped the rewrite on the deploy preview of fluent/fluentd-docs-gitbook#710. The site already uses directory_indexes, so other links should not change.

🤖 Generated with Claude Code

Pretty URLs rewrites links whose path is /index.html, even when they
point to other hosts. The calyptia-fluentd package browser links on the
download page lose index.html and open an XML listing instead. The same
setting fixed this on the deploy preview of fluent/fluentd-docs-gitbook.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
@netlify

netlify Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for fluentd-website ready!

Name Link
🔨 Latest commit e989770
🔍 Latest deploy log https://app.netlify.com/projects/fluentd-website/deploys/6ac74100b54cc900086bfceb
😎 Deploy Preview https://deploy-preview-521--fluentd-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Watson1978
Watson1978 requested a review from kenhys October 8, 2026 07:07
@kenhys
kenhys merged commit abd8979 into fluent:master Oct 8, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants