-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Filter Slackbot crawler #5284
Comments
+1 |
The three Slack user agents for their crawlers are detailed at https://api.slack.com/robots. |
If we're gonna do this, we might as well also get Facebook and Twitter. Unless we already do. (Not looking at code yet) |
Facebook is in the list of crawlers at https://github.com/getsentry/sentry/blob/master/src/sentry/filters/web_crawlers.py |
Cool, then let's add Slack and Twitter to this. I can do this at some point unless you want to throw up a PR for it. <3 |
@mattrobenolt it seemed easy so I went ahead and put up a PR - #5285. |
We get issues reported in Sentry whenever we post our internal app links in Slack because the Slackbot web crawler tries to load them and fails. Would it be possible to get Slackbot added to the list of crawlers that are ignored in Sentry?
The text was updated successfully, but these errors were encountered: