Skip to content

feat(offline-dynamodb-streams): table does not have streams error#98

Closed
dolsem wants to merge 1 commit into
CoorpAcademy:masterfrom
dolsem:master
Closed

feat(offline-dynamodb-streams): table does not have streams error#98
dolsem wants to merge 1 commit into
CoorpAcademy:masterfrom
dolsem:master

Conversation

@dolsem

@dolsem dolsem commented Oct 23, 2019

Copy link
Copy Markdown

Currently if a table without streams is provided to serverless-offline-dynamodb-streams, it crashes with MissingRequiredParameter: Missing required key 'StreamArn' in params. This PR adds a more descriptive error.

@sergio-bele

Copy link
Copy Markdown

How can I fix the error? do you have some code examples?

silouone added a commit that referenced this pull request Jun 19, 2026
…no-streams fixes

- Serverless v4 logger migration (src/log.js shim threaded in; handler errors are now logged).
- Fix undefined this.region -> this.options.region so emitted events carry awsRegion. Re-implements #166 (thanks @zlalvani).
- Throw a clear 'Table <name> does not have streams enabled' error instead of failing cryptically. Re-implements #98 (thanks @dolsem).
- Rename the copy-pasted KinesisEvent class to DynamodbStreamsEvent.
- Backward compatible with serverless v3. Adds docker-free unit tests. v7.1.0.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@silouone

Copy link
Copy Markdown
Member

Thanks @dolsem for surfacing a clear "table does not have streams" error instead of the cryptic crash users used to hit — a much nicer developer experience. This is now implemented in #265 (on master now, released shortly in the next release of serverless-offline-dynamodb-streams), which also brings Serverless Framework v4 support while keeping our test/CI harness on osls (the community license-free fork of Serverless v3) so local-dev and CI stay free and open after v4 moved to a paid license. Closing in favor of #265 — thank you 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants