Skip to content

Conversation

r3dg33k
Copy link

@r3dg33k r3dg33k commented Aug 13, 2025

You're A Rockstar

Thank you for submitting a Pull Request (PR) to the Cheat Sheet Series.

🚩 If your PR is related to grammar/typo mistakes, please double-check the file for other mistakes in order to fix all the issues in the current cheat sheet.

Please make sure that for your contribution:

  • In case of a new Cheat Sheet, you have used the Cheat Sheet template.
  • All the markdown files do not raise any validation policy violation, see the policy.
  • All the markdown files follow these format rules.
  • All your assets are stored in the assets folder.
  • All the images used are in the PNG format.
  • Any references to websites have been formatted as [TEXT](URL)
  • You verified/tested the effectiveness of your contribution (e.g., the defensive code proposed is really an effective remediation? Please verify it works!).
  • The CI build of your PR pass, see the build status here.

If your PR is related to an issue, please finish your PR text with the following line:

This PR fixes issue #<REPLACE WITH ISSUE NUMBER>.

Thank you again for your contribution 😃

jmanico
jmanico previously approved these changes Aug 13, 2025
@jmanico
Copy link
Member

jmanico commented Aug 25, 2025

I'm not 100% sure on this one, @mackowski ?

@mackowski
Copy link
Collaborator

hey @righettod can you help with this PR? :)

@jmanico
Copy link
Member

jmanico commented Aug 28, 2025

I tend to be thumbs-up on this PR. It seems reasonable to want to AI crawlers of the day to leave us alone. :)

@mackowski
Copy link
Collaborator

@jmanico the problem is that the recommendation is from different header
Screenshot 2025-09-08 at 12 34 56
Under X-Robots-Tag we will have:
'Disable sending this header. To remove the X-AspNetMvc-Version header, add the below line in Global.asax file.'

Copy link
Collaborator

@mackowski mackowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update the 'Recommendation' section

@jmanico
Copy link
Member

jmanico commented Sep 11, 2025

I'm with you now @mackowski

@mackowski mackowski requested a review from Copilot October 3, 2025 12:31
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds documentation for the X-Robots-Tag HTTP header to the HTTP Headers Cheat Sheet. The X-Robots-Tag header is used to control how search engines and AI bots index and display non-webpage content.

  • Added a new section documenting the X-Robots-Tag header with description and example usage
  • Included a note about crawler behavior and compliance
  • Added a recommendation section for the header

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

mackowski and others added 2 commits October 3, 2025 14:37
Added explanation for X-DNS-Prefetch-Control header.
mackowski
mackowski previously approved these changes Oct 3, 2025
@mackowski
Copy link
Collaborator

@jmanico @szh I updated this PR (with copilot) to fix previous issue

@mackowski mackowski requested review from jmanico and szh October 3, 2025 12:41
@szh
Copy link
Collaborator

szh commented Oct 3, 2025

I'm still not 100% convinced we should outright recommend disallowing crawlers without caveats. There are downsides. For example, I would argue we definitely want AI to ingest the cheat sheets so they will be more likely to give good recommendations for people who will inevitably ask LLMs for security advice.

@mackowski
Copy link
Collaborator

Agree, we should add this header but think how to describe the recommendation.
By the way - on the pages where I want to disable crawler it is better to implement solid AuthN/Z ;)

@jmanico
Copy link
Member

jmanico commented Oct 8, 2025

Overall we want this recommendation for any data we want to protect from crawlers. It's totally fair that public data does not need this.

Clarified the explanation of the X-Robots-Tag header and its usage. Improved formatting and notes for better readability.
Corrected formatting and removed redundant text in the HTTP Headers Cheat Sheet.
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.

5 participants