Skip to content

Feature/dify improvement#118

Open
takakotucker wants to merge 56 commits intoaws-samples:mainfrom
kodato-dev:feature/dify-improvement
Open

Feature/dify improvement#118
takakotucker wants to merge 56 commits intoaws-samples:mainfrom
kodato-dev:feature/dify-improvement

Conversation

@takakotucker
Copy link
Copy Markdown

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

shogo-sonoda and others added 30 commits July 8, 2025 14:16
- 環境変数からIPアドレスと国コードを読み取る機能を実装
- EnvironmentPropsにallowedCountryCodesプロパティを追加
- WAF設定ガイドとドキュメントを追加・更新

環境変数: ALLOWED_IPV4_CIDRS, ALLOWED_IPV6_CIDRS, ALLOWED_COUNTRY_CODES
- WAF_SETUP.mdとwaf-config-examples.mdを1つのドキュメントに統合
- 重複した内容を削除し、より簡潔で分かりやすい構成に改善
- Quick Start、設定例、リファレンス、ベストプラクティスの流れで整理
- 英語に統一してグローバルユーザーにも対応

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
WAF設定を環境変数で管理
docs: デプロイ手順の追記
- CDK_DEFAULT_ACCOUNTのエクスポートを修正
- ALLOWED_IPV4_CIDRSのデフォルト値を削除し、環境変数からの読み取りに変更
fix: 環境変数の設定を修正
セキュリティ強化
  - dev/main ブランチへのプッシュ時に自動でCDKデプロイを実行
  - 環境別のAWS認証情報をGitHub Secretsで管理
  - dev環境: AWSアカウント 058264089195
  - prod環境: AWSアカウント 478681634885
  - ハードコードされていたAWSアカウントIDをGitHub Secretsに移行
  - deploy.ymlのセキュリティを向上
  - コメント内のアカウントIDを部分的にマスク(先頭5文字のみ表示)

  必要なGitHub Secrets:
  - AWS_ACCOUNT_ID_DEV
  - AWS_ACCOUNT_ID_PROD
  - 環境別シークレット名を統一(_DEV/_PROD サフィックスを削除)
  - GitHub Environments(dev/main)を使用してシークレットを管理
  - PR作成・更新時に自動でCDK diffを実行
  - ターゲットブランチ(dev/main)の環境と差分を比較
  - diff結果をPRコメントに自動投稿(既存コメントは更新)
GitHub Variablesを使用してALLOWED_IPV4_CIDRSとALLOWED_COUNTRY_CODESを
環境変数として設定できるように修正しました。

- cdk-workflow.yml: CDK diff実行時に環境変数を設定
- deploy.yml: CDKデプロイ時に環境変数を設定
- deployment.md: GitHub Variables設定手順を追加

これにより、IP制限と国コード制限の設定をGitHub UIから管理できるようになります。

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
GitHub Environmentsの名前をmainからprodに変更し、
本番環境をより明確に識別できるようにしました。

- cdk-workflow.yml: PRターゲットブランチがmainの場合はprod環境を使用
- deploy.yml: mainブランチへのpushはprod環境にデプロイ
- deployment.md: ドキュメントの環境名をprodに更新

これにより、開発(dev)と本番(prod)の環境が明確に区別されます。

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
GitHub Actionsのワークフローで、CDK diffの出力をBase64エンコードしてGITHUB_OUTPUTに保存するように修正しました。これにより、複数行の出力を適切に処理できるようになります。

- cdk-workflow.yml: 出力変数の保存方法を変更し、Base64エンコードを適用
parseIpAddresses関数内のコードのインデントを整え、if文の条件を複数行に分割しました。これにより、コードの可読性が向上します。
 feat: GitHub ActionsによるCDK自動デプロイワークフローを追加
GitHub Actions の導入
allowedCountryCodesの無効化
Revert "allowedCountryCodesの無効化"
shogo-sonoda and others added 18 commits September 30, 2025 12:09
chore: upstream/mainからの変更を取り込み (Dify 1.9.1)
feat: add aws-marketplace:ViewSubscriptions permission to API service…
feat: add aws-marketplace:Subscribe permission to API service role
fix: increase WORKER_TIMEOUT environment variable from 15 to 180 seconds
フォーク元リポジトリの最新変更をマージ
IP addresses and country codes are not sensitive data, so they should
be stored in Variables for easier debugging and management.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
refactor: use GitHub Variables instead of Secrets for IP restrictions
- CPU 105%, memory 99.76% confirmed via Container Insights
- Root cause: API task undersized (1vCPU/2GB) with multiple containers competing
- Recommended fix: upsize to 2vCPU/4GB + Redis t4g.micro -> t4g.small
- Update CDK snapshots (WORKER_TIMEOUT and aws-marketplace IAM permissions)
- WORKER_TIMEOUT 15s -> 180s (12x longer CPU hold per task)
- Dify upgrade to 1.11.1
- PluginDaemon upgrade
- Trigger/Webhook feature addition
@yhiroaky
Copy link
Copy Markdown
Contributor

yhiroaky commented Apr 6, 2026

Hi @takakotucker, thank you for your contribution.

Is this PR routed correctly? If yes, please include the details.

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.

4 participants