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

Don't index deeplinks #45

Open
MattIPv4 opened this issue Aug 6, 2019 · 2 comments
Open

Don't index deeplinks #45

MattIPv4 opened this issue Aug 6, 2019 · 2 comments
Labels
💻 Client-side 🌐 DNS Lookup Relating to the DNS lookup tool 💙 Enhancement good first issue Good for newcomers help wanted Extra attention is needed ✉️ SPF Explainer Relating to the SPF explainer tool

Comments

@MattIPv4
Copy link
Contributor

MattIPv4 commented Aug 6, 2019

We need to ensure we do not allow indexing of deeplinks in search engines.

Eg. We can index the main site: github.io/dns-tool/ but we do not want to index github.io/dns-tool/?domain=somedodgy.site

@IAmJSD
Copy link
Contributor

IAmJSD commented Aug 8, 2019

I don't think there is a way that this can be an issue. I believe this is all done on Google's side. With that being said, we should do some precautions since I don't know how trivial it is to force Google to index a page. I believe this has to be done by someone with access to the root directory though according to this documentation.

@MattIPv4 MattIPv4 added ✉️ SPF Explainer Relating to the SPF explainer tool 🌐 DNS Lookup Relating to the DNS lookup tool labels Sep 12, 2019
@MattIPv4
Copy link
Contributor Author

I believe Google will wait for some JS to run when loading a page, so we could feasibly ensure we add a noindex meta tag if we detect a query param (or rather, whenever we have a non-default state, which gets stored as a query param). Not sure it'd be perfect, but it would be better than nothing.

@MattIPv4 MattIPv4 added help wanted Extra attention is needed good first issue Good for newcomers labels Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 Client-side 🌐 DNS Lookup Relating to the DNS lookup tool 💙 Enhancement good first issue Good for newcomers help wanted Extra attention is needed ✉️ SPF Explainer Relating to the SPF explainer tool
Projects
None yet
Development

No branches or pull requests

2 participants