Skip to content

feat: Cross-region SNS subscription#102

Merged
nekketsuuu merged 1 commit intomasterfrom
nekketsuuu-sns-cross-region
Oct 23, 2025
Merged

feat: Cross-region SNS subscription#102
nekketsuuu merged 1 commit intomasterfrom
nekketsuuu-sns-cross-region

Conversation

@nekketsuuu
Copy link
Contributor

Amazon SNS supports cross-region subscriptions: https://docs.aws.amazon.com/sns/latest/dg/sns-cross-region-delivery.html This patch added multi-region support to the SNS client, allowing cross-region subscriptions to be created from the Barbeque web console.

At the same time, this patch introduced a plain text input box for SNS topic ARNs rather than a select box, because:

  1. A list of topic ARNs in multiple regions will be long, and
  2. The list of topic ARNs in the default region is already long in Cookpad.

Screenshots:

Before After
screenshot screenshot

Amazon SNS supports cross-region subscriptions: https://docs.aws.amazon.com/sns/latest/dg/sns-cross-region-delivery.html
This patch added multi-region support to the SNS client, allowing cross-region subscriptions to be created from the Barbeque web console.

At the same time, this patch introduced a plain text input box for SNS topic ARNs rather than a select box, because:

1. A list of topic ARNs in multiple regions will be long, and
2. The list of topic ARNs in the default region is already long in Cookpad.
@nekketsuuu nekketsuuu requested a review from a team October 22, 2025 09:16
@nekketsuuu
Copy link
Contributor Author

Thanks!

@nekketsuuu nekketsuuu merged commit 72577fa into master Oct 23, 2025
8 checks passed
@nekketsuuu nekketsuuu deleted the nekketsuuu-sns-cross-region branch October 23, 2025 02:38
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