Skip to content

feat(feishu): add custom domain configuration option#412

Open
chenhg5 wants to merge 1 commit intomainfrom
feat/issue-402-feishu-custom-domain
Open

feat(feishu): add custom domain configuration option#412
chenhg5 wants to merge 1 commit intomainfrom
feat/issue-402-feishu-custom-domain

Conversation

@chenhg5
Copy link
Copy Markdown
Owner

@chenhg5 chenhg5 commented Apr 1, 2026

Summary

  • Add optional domain parameter for Feishu/Lark platforms
  • Support custom Feishu deployments with custom domain URLs
  • Default behavior unchanged when domain is not specified

Usage

[[projects.platforms]]
type = "feishu"
[projects.platforms.options]
app_id = "cli_xxx"
app_secret = "secret"
domain = "https://custom.feishu.cn"  # optional

Test plan

  • go build ./... passes
  • go vet ./... passes
  • go test ./platform/feishu/... passes (all tests including new custom domain tests)
  • Manual smoke test: configure custom domain and verify API calls use correct endpoint

Closes #402

Add optional `domain` parameter for Feishu/Lark platforms to support
custom Feishu deployments. The underlying SDK already supports custom
domains, but the configuration layer didn't expose this capability.

Usage in config.toml:
  [[projects.platforms]]
  type = "feishu"
  [projects.platforms.options]
  app_id = "cli_xxx"
  app_secret = "secret"
  domain = "https://custom.feishu.cn"  # optional

Default behavior is unchanged - Feishu uses open.feishu.cn and Lark
uses larksuite.com when domain is not specified.

Closes #402

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@chenhg5
Copy link
Copy Markdown
Owner Author

chenhg5 commented Apr 1, 2026

LGTM! Clean implementation of custom domain configuration.

✅ Feature correctness:

  • Adds optional domain parameter for Feishu/Lark platforms
  • Properly handles whitespace trimming
  • Default behavior unchanged when domain not specified

✅ Tests: Added TestNewFeishu_CustomDomain and TestNewLark_CustomDomain covering the feature.

✅ CI: All checks pass.

Ready for merge.

@chenhg5
Copy link
Copy Markdown
Owner Author

chenhg5 commented Apr 2, 2026

⚠️ Merge Conflict

This PR has merge conflicts with the main branch. Please resolve the conflicts:

git fetch origin
git checkout feat-feishu-custom-domain
git merge origin/main
# Resolve conflicts, then:
git push

Once resolved, the PR can proceed to review and merge.

Copy link
Copy Markdown
Owner Author

@chenhg5 chenhg5 left a comment

Choose a reason for hiding this comment

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

此 PR 有 merge conflicts,需要解决后再合并。

请 rebase 到最新的 main 分支并解决冲突。

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.

[Feature] 是否支持修改飞书的域名?

2 participants