Skip to content

docs(bunny): document IP header trust assumptions and security considerations#185

Closed
Copilot wants to merge 3 commits intofeat/bunny-adapterfrom
copilot/sub-pr-182
Closed

docs(bunny): document IP header trust assumptions and security considerations#185
Copilot wants to merge 3 commits intofeat/bunny-adapterfrom
copilot/sub-pr-182

Conversation

Copy link

Copilot AI commented Feb 17, 2026

Added documentation clarifying that the Bunny.net adapter assumes the platform filters x-forwarded-for and x-real-ip headers to prevent IP spoofing, but this has not been verified against official documentation.

Changes:

  • Added inline comment in src/adapters/bunny.ts explaining IP header trust assumptions
  • Documented comparison to similar edge runtimes (Cloudflare's cf-connecting-ip)
  • Added warning to verify behavior before security-critical use (rate limiting, access control)
// IP address from Bunny headers
// Note: This implementation assumes Bunny.net Edge Scripting filters and sets
// x-forwarded-for and x-real-ip headers to prevent IP spoofing. While this
// follows the pattern used in similar edge runtimes (like Cloudflare with
// cf-connecting-ip), we have not verified this behavior in Bunny.net's
// official documentation. Users should verify that Bunny.net's infrastructure
// properly sanitizes these headers before relying on them for security-critical
// purposes such as rate limiting or access control.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 17, 2026 08:36
… needs

Co-authored-by: pi0 <5158436+pi0@users.noreply.github.com>
Co-authored-by: pi0 <5158436+pi0@users.noreply.github.com>
Copy link
Author

Copilot AI commented Feb 17, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • docs.bunny.net
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)
  • support.bunny.net
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)
  • www.npmjs.com
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] WIP Addressing documentation feedback on Bunny.net adapter docs(bunny): document IP header trust assumptions and security considerations Feb 17, 2026
Copilot AI requested a review from pi0 February 17, 2026 08:39
@pi0 pi0 closed this Feb 17, 2026
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

Comments