search

sst / sst

Build full-stack apps on your own infrastructure.
https://sst.dev
MIT License
24.05k stars 1.85k forks source link

fix dynamodb subscription on existing tables #5908

Open santiperone opened 3 weeks ago

santiperone commented 3 weeks ago

This fixes an issue where this.isStreamEnabled won't be set when referencing existing tables .

https://github.com/sst/sst/blob/8a82acf81b5ab681899b2168385b8c49aa443bc1/platform/src/components/aws/dynamo.ts#L621-L626

I guess another alternative would be to check against nodes.table.streamArn or nodes.table.streamEnabled in that line instead, lmk if you want me to change that.

santiperone commented 3 weeks ago

Actually after a second look using "nodes.table.streamEnabled" just seems more reliable.