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

Update Sitemap URLs to Use Valid Paths in sitemap.mjs #1861

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

Conversation

rbpi
Copy link

@rbpi rbpi commented Jul 16, 2024

There are errors in the sitemap.xml file generated by default. It can visit https://gchq.github.io/CyberChef/sitemap.xml to see the paths of all URLs that are missing from the CyberChef directory.

This PR updates the URLs in the sitemap to ensure they use valid paths, specifically updating the base path to include "CyberChef" where necessary.

Changes include:

  • Line 18: Updating the root URL path from "/" to "/CyberChef/".
  • Line 25: Modifying operation URLs from /?op=... to /CyberChef/?op=....

These changes will help search engines properly index the CyberChef operations and improve the overall SEO of the site. This ensures that all generated sitemap URLs are accessible and correctly formatted.

Please review the changes. Thank you!

@CLAassistant
Copy link

CLAassistant commented Jul 16, 2024

CLA assistant check
All committers have signed the CLA.

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