Skip to content

feat(discord): add proxy support and tests#853

Open
nayihz wants to merge 1 commit intosipeed:mainfrom
nayihz:feat_discord_proxy
Open

feat(discord): add proxy support and tests#853
nayihz wants to merge 1 commit intosipeed:mainfrom
nayihz:feat_discord_proxy

Conversation

@nayihz
Copy link
Contributor

@nayihz nayihz commented Feb 27, 2026

📝 Description

Add Discord proxy support for both REST and websocket paths, with coverage tests. This ensures Discord can work in proxied environments when a channel proxy is configured (including socks5://) and improves behavior around proxy setup validation.

🗣️ Type of Change

  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 📖 Documentation update
  • ⚡ Code refactoring (no functional changes, no api changes)

🤖 AI Code Generation

  • 🤖 Fully AI-generated (100% AI, 0% Human)
  • 🛠️ Mostly AI-generated (AI draft, Human verified/modified)
  • 👨‍💻 Mostly Human-written (Human lead, AI assisted or none)

🔗 Related Issue

N/A

📚 Technical Context (Skip for Docs)

  • Reference URL: N/A
  • Reasoning: Route Discord HTTP client and websocket dialer through configured proxy logic, and add tests to prevent regressions.

🧪 Test Environment

  • Hardware: PC
  • OS: macOS
  • Model/Provider: N/A
  • Channels: Discord

📸 Evidence (Optional)

Click to view Logs/Screenshots

Added pkg/channels/discord_test.go and validated proxy setup behavior via unit tests.

☑️ Checklist

  • My code/docs follow the style of this project.
  • I have performed a self-review of my own changes.
  • I have updated the documentation accordingly.

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.

1 participant