flightstats / hub

fault tolerant, highly available service for data storage and distribution
http://www.flightstats.com
MIT License
103 stars 35 forks source link

Fix webhook config dynamo scans #1235

Closed lkemmerer closed 4 years ago

lkemmerer commented 4 years ago

This adds the fix to the bug that Lucas discovered around webhook config retrieval. The bug was that once the returned list of records reached a certain size and required paging, the method would enter an infinite loop.

Since I was adding tests (based on channel config dao tests), I pulled over a fix that Damon made to channel configs, where improperly constructed dynamo records are dropped rather than throwing an error and causing the hub to behave very badly.