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

netlify block #18758

Merged
merged 28 commits into from
Jul 30, 2024
Merged

netlify block #18758

merged 28 commits into from
Jul 30, 2024

Conversation

ebembi-crdb
Copy link
Contributor

@ebembi-crdb ebembi-crdb commented Jul 23, 2024

Block Traffic from Bytedance Bots to Prevent Bandwidth Overages

This solves - https://cockroachlabs.atlassian.net/browse/DOC-10684

Copy link

github-actions bot commented Jul 23, 2024

Files changed:

  • src/current/netlify.toml
  • src/current/netlify/edge-functions/blockBytedance.ts
  • src/current/package.json

Copy link

netlify bot commented Jul 23, 2024

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit e50451e
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-interactivetutorials-docs/deploys/66a8936e4a982d000801be63

Copy link

netlify bot commented Jul 23, 2024

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit e50451e
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-api-docs/deploys/66a8936ef1d999000863ee98

Copy link

netlify bot commented Jul 23, 2024

Netlify Preview

Name Link
🔨 Latest commit e50451e
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-docs/deploys/66a8936f00af6f00086e9a31
😎 Deploy Preview https://deploy-preview-18758--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ebembi-crdb
Copy link
Contributor Author

Screenshot 2024-07-23 at 8 31 10 PM Tested using bytedance user agent

@ebembi-crdb ebembi-crdb requested a review from a team as a code owner July 24, 2024 19:39
Copy link
Contributor

@shannonbradshaw shannonbradshaw left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

Let's wait until Monday to deploy.

@shannonbradshaw
Copy link
Contributor

@ebembi-crdb we can go live with this at any point, when you have a chance to monitor and revert if we break something.

Copy link
Contributor

@shannonbradshaw shannonbradshaw left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@ebembi-crdb ebembi-crdb merged commit 75c2d98 into main Jul 30, 2024
6 checks passed
@ebembi-crdb ebembi-crdb deleted the feat/netlify-block branch July 30, 2024 09:07
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