Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: ListSubscriptionsByTopic no longer paginates results #10796

Closed
1 task done
matt-phylum opened this issue May 8, 2024 · 2 comments · Fixed by #10810
Closed
1 task done

bug: ListSubscriptionsByTopic no longer paginates results #10796

matt-phylum opened this issue May 8, 2024 · 2 comments · Fixed by #10810
Assignees
Labels
aws:sns Amazon Simple Notification Service status: in progress Currently being worked on type: bug Bug report

Comments

@matt-phylum
Copy link

matt-phylum commented May 8, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Regardless of how many subscribers you have, ListSubscriptionsByTopic returns all of them in a single page.

Expected Behavior

ListSubscriptionsByTopic should paginate results, similar to how AWS does it and previous versions of LocalStack did it, so that tests of what happens when there are multiple pages work correctly.

How are you starting LocalStack?

With a docker-compose file

Steps To Reproduce

Localstack is started as part of a custom test environment. 101 e-mails are registered via Subscribe, and then the subscriptions are retrieved using ListSubscriptionsByTopic.

Environment

- OS: localstack/localstack container
- LocalStack: localstack/localstack@latest (sha256:f9310db87d9d76c449918a849562303b1c53f77c3598fdcb9c6cd7843d0f8cb6?)

Anything else?

This was working yesterday (2024-05-07). It looks like #10788 replaced the existing implementation that supported pagination with a new one that doesn't. ListSubscriptions is probably similarly affected.

@matt-phylum matt-phylum added status: triage needed Requires evaluation by maintainers type: bug Bug report labels May 8, 2024
@localstack-bot
Copy link
Collaborator

Welcome to LocalStack! Thanks for reporting your first issue and our team will be working towards fixing the issue for you or reach out for more background information. We recommend joining our Slack Community for real-time help and drop a message to LocalStack Pro Support if you are a Pro user! If you are willing to contribute towards fixing this issue, please have a look at our contributing guidelines and our contributing guide.

@bentsku bentsku self-assigned this May 8, 2024
@bentsku bentsku added aws:sns Amazon Simple Notification Service status: in progress Currently being worked on and removed status: triage needed Requires evaluation by maintainers labels May 8, 2024
@bentsku
Copy link
Contributor

bentsku commented May 8, 2024

Hello @matt-phylum and thanks for the report!

Yes, this bug fix for cross-account subscriptions introduced the small change in behavior, I'll re-introduce the pagination shortly. Thanks again for the report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aws:sns Amazon Simple Notification Service status: in progress Currently being worked on type: bug Bug report
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants