-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Add noindex
flag to API repsonse
#7822
Conversation
The usefulness of this data when it's incomplete for remote users is arguable. I think for this to be done completely, the property must become part of the ActivityPub actor (and therefore move from settings to accounts table?). The property would also be a new addition to AP vocabulary and therefore warrants extended discussion. |
yes, this is incomplete for remote users. however, in some cases, it is really useful. for example, auto opt-out from search engine that crawl only local timeline. |
@rinsuki what search engines crawl the public timelines? I'm also concerned about remote users, but maybe it's safe if we always give noindex values of |
@@ -48,6 +48,11 @@ def header_static | |||
full_asset_url(object.header_static_url) | |||
end | |||
|
|||
def noindex | |||
user = object.user | |||
user.settings['noindex'] if user |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return true unless object.user
object.user.settings['noindex']
@nightpool for example, http://mastodonsearch.jp/, https://tootsearch.chotto.moe/, etc crawls public timeline. |
tootsearch reads FTL. |
The |
Maybe. But |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
ref: #6361
if not local user, return value is null.