Skip to content
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Revert "bugfix for sns topic deletion upon termination"
This reverts commit bcaa9bb.
  • Loading branch information
Amit Mor committed Jul 23, 2021
commit 62ac6b01d3a1fe12e5acd28cf556cf751b78fd7a
9 changes: 6 additions & 3 deletions pubsub/aws/snssqs/snssqs.go
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ func NewSnsSqs(l logger.Logger) pubsub.PubSub {
return &snsSqs{
logger: l,
subscriptions: []*string{},
pattern: regexp.MustCompile("[^a-zA-Z0-9_\\-]+"),
}
}

Expand Down Expand Up @@ -221,7 +220,7 @@ func (s *snsSqs) Init(metadata pubsub.Metadata) error {
}

func (s *snsSqs) createTopic(topic string) (string, string, error) {
hashedName := s.nameToValidName(topic)
hashedName := nameToHash(topic)
createTopicResponse, err := s.snsClient.CreateTopic(&sns.CreateTopicInput{
Name: aws.String(hashedName),
Tags: []*sns.Tag{{Key: aws.String(awsSnsTopicNameKey), Value: aws.String(topic)}},
Expand Down Expand Up @@ -492,7 +491,11 @@ func (s *snsSqs) Subscribe(req pubsub.SubscribeRequest, handler pubsub.Handler)
}

func (s *snsSqs) Close() error {
s.logger.Debugf("Closing sns-sqs pubsub component. This is NOOP")
for _, sub := range s.subscriptions {
s.snsClient.Unsubscribe(&sns.UnsubscribeInput{
SubscriptionArn: sub,
})
}

return nil
}
Expand Down