ziggy42 / ontopic

Display SNS messages on your terminal
Apache License 2.0
23 stars 3 forks source link

Enhance cleanup mechanism to account for token expiry #1

Open whummer opened 4 years ago

whummer commented 4 years ago

Thanks for publishing this useful library!

After staying connected for some time, the SQS polling fails with:

ExpiredToken: The security token included in the request is expired

In this case, the Node.js process dies, but is unable to perform the cleanup (because deleting the SQS queues would require a valid token).

It would be useful to have a more robust cleanup mechanism, e.g., by defining the name of the queue via the command line, or based on certain attributes attached to the test queue. ontopic could then identify and clean up old left-over SQS queues on startup.

Another option could be to allow the user to update the credentials at runtime.

ziggy42 commented 4 years ago

Good point, I will need to think about what we could realistically do. In the meantime I added some improvements in https://github.com/ziggy42/ontopic/pull/3: