Skip to content

fix: avoid rewriting existing IndexNow key files#11

Merged
fidalgo merged 1 commit into
mainfrom
fix-rewrite-key
May 1, 2026
Merged

fix: avoid rewriting existing IndexNow key files#11
fidalgo merged 1 commit into
mainfrom
fix-rewrite-key

Conversation

@fidalgo
Copy link
Copy Markdown
Contributor

@fidalgo fidalgo commented May 1, 2026

Make IndexNow key-file handling explicit and less invasive.

When a configured IndexNow key is present, sitemap creation still ensures the matching verification file exists by default. If the file is already present and valid, Indexmap now leaves it unchanged instead of rewriting it. This keeps the simple public/ workflow intact while avoiding unnecessary filesystem churn.

Add config.index_now.write_key_file so applications with custom serving or staging flows can opt out of automatic key-file writes entirely. The configured key remains available for IndexNow pinging even when file writing is disabled.

This supports apps that generate sitemap XML in a temporary staging directory while serving the IndexNow verification file from a separate public path.

How to test

Release notes

Does this change need an explicit changelog or release-note callout beyond the PR title?

Checklist

  • PR title is a Conventional Commit and suitable for squash merge
  • Tests added/updated (if applicable)
  • Docs updated (if applicable)
  • I kept changes focused and easy to review
About the maintainers

Built by Ethos Link, the team behind Reviato.
Capture. Interpret. Act.
Turn guest feedback into clear next steps for your team. Collect private appraisals, spot patterns across reviews, and act before small issues turn into public ones.

Make IndexNow key-file handling explicit and less invasive.

When a configured IndexNow key is present, sitemap creation still
ensures the matching verification file exists by default. If the file is
already present and valid, Indexmap now leaves it unchanged instead of
rewriting it. This keeps the simple public/ workflow intact while
avoiding unnecessary filesystem churn.

Add config.index_now.write_key_file so applications with custom serving
or staging flows can opt out of automatic key-file writes entirely. The
configured key remains available for IndexNow pinging even when file
writing is disabled.

This supports apps that generate sitemap XML in a temporary staging
directory while serving the IndexNow verification file from a separate
public path.
@fidalgo fidalgo merged commit 4fd5b3b into main May 1, 2026
4 checks passed
@fidalgo fidalgo deleted the fix-rewrite-key branch May 1, 2026 18:47
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.

1 participant